-
Notifications
You must be signed in to change notification settings - Fork 23.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Stop appending '/api' to galaxy server url #63238
Conversation
Since not all galaxy REST api server URLs live at '/api', stop always appending it to the 'url' value loaded from config.
Since we retry now with an added /api/, mock it as well.
All unit tests are passing. One test is marked as |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shipit
@alikins Oh, I overlooked that this could use a changelog. |
@alikins @chouseknecht Merged. Please add a changelog when you open the backport to stable-2.9 and then I'll get the backpotr merged tomorrow. Thanks! |
* Stop appending '/api' to galaxy server url (#63238) * Stop appending '/api' to configured galaxy server url Since not all galaxy REST api server URLs live at '/api', stop always appending it to the 'url' value loaded from config. * Add note about manually migrated galaxy configs and /api * Add '/api/' to galaxy url and guessing if galaxy API * Fix most unit tests (update to expect /api/) * Fix test_initialise_unknown unit test Since we retry now with an added /api/, mock it as well. * Update fallback default avail_ver to new format (cherry picked from commit bad7269) * Add changelog fragment galaxy_api_config * Fix galaxy url use everywhere when url is set in config. (#63286) In addition to trying the configured url (for ex, a migrated 'https://galaxy.ansible.com/') there is an attempt at that URL with '/api' postpended. If the extended URL works, update GalaxyAPI.api_server to use the extended URL. Previously it only used it for finding the API root info ('available_versions', etc) (cherry picked from commit ed203c5)
SUMMARY
Stop appending '/api' to configured galaxy server url. Since not all Galaxy REST api server URLs live at
/api
, the client can no longer blindly append/api
to the URL found in the config.ISSUE TYPE
COMPONENT NAME
ansible-galaxy
ADDITIONAL INFORMATION