Try connecting to Ubuntu keyserver on port 80 if default HKP port fails

[Richard Höchenberger (hoechenberger)]

Closes #719
Closes #717

cc Samruddhi Khandale (@samruddhikhandale)

[Richard Höchenberger (hoechenberger)]

This is ready for review.

[Samruddhi Khandale (samruddhikhandale)]

Richard Höchenberger (@hoechenberger) did this change work with your local config/machine? The tests are passing, so should be safe to merge it in. Also, I'll shortly test these changes on my local machine.

Left one minor comment.

[Samruddhi Khandale (samruddhikhandale)]

Richard Höchenberger (@hoechenberger) did this change work with your local config/machine?

I saw your PR description update, looks like this PR changes are working for you? 🎉

[Richard Höchenberger (hoechenberger)]

Samruddhi Khandale (@samruddhikhandale) I was struggling to get the tests to work on my corporate machine because I need to inject custom SSL certificates. I have written a custom feature to do that, but didn't have time to modify the tests to actually use this feature. So my honest answer is: no, I could not fully run the test suite yet. LMK if you would still like to go ahead and merge (after which I'd be able to test in production) and we'll have to revert if necessary; or otherwise I'll have to get the test suite to run locally, but it may take a couple of days (as I'm very busy today and probably also tomorrow)

[Samruddhi Khandale (samruddhikhandale)]

Richard Höchenberger (@hoechenberger) I have a simpler way to test these changes, can you check if you can create a dev container with https://github.com/samruddhikhandale/test-gpg-keyserver/tree/main ?

It adds the same dev configuration as mentioned in #717, the only difference is that it doesn't pull the python Feature from ghcr, instead it uses a local Feature (which has these ^ PR changes)

It would be great to know if this changes fixes your issue before we roll out this change. It works good with my testing, however, the gpg issue behaves differently on different machines (don't want to accidentally cause Feature failure blips - given that this PR updates seven Features)

[Richard Höchenberger (hoechenberger)]

Samruddhi Khandale (@samruddhikhandale) Thank you! I will test this on my macOS machine and ask colleagues of mine to test on Linux and Windows too. I will get back to you once I have the test results.

[Richard Höchenberger (hoechenberger)]

Samruddhi Khandale (@samruddhikhandale) Tested and working on two separate macOS computers. I couldn't test on other operating systems, unfortunately!

[Samruddhi Khandale (samruddhikhandale)]

Looks good to me~ Thank you!