Use HTTPS by default and release v1.2.0

[slint]

No description provided.

[slint]

@lnielsen not sure if it should be v2.0.0 actually... One can see it as a "security fix", but on the other hand changing default parameters is breaking API...

[fenekku]

Coordination:

This was put on the community PR board, so I guess I have to coordinate this 😆 ... So want to rebase this and get it in? @slint @lnielsen

[slint]

I think the issue with this one is that it's a pretty big "breaking change" to switch to HTTPS URL generation, affecting many other tests e.g. on RDM REST API responses and serialized formats. This means that we'll have to first check if we have upper pins for idutils in other modules, to avoid having a wave of breaking tests.

[chokribr]

Coordination: @lnielsen @slint shall we move this PR to To Review or ask @fenekku for a second reviewer aprovement to aprove!

[fenekku]

It looks good to me.

My vote would be to make a ticket in invenio-app-rdm to

• cap idutils in previous versions of invenio-app-rdm and related modules if not already
• then release this
• then integrate the new version of idutils in invenio-app-rdm and related modules

For me: better security / up-to-date schemes trumps potential backward incompatibility / small breakage. And moving sooner rather than later is better. I don't have commit/release rights on this repo anyway though.