-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
fix(manager/git-submodule): use appropriate hostType for auth #17400
Conversation
@rarkins should we propably need to change this Line 42 in a5b9ad0
So if |
Don't you agree the same fix could apply to the GitRefsDatasource and GitTagsDatasource? |
that's not so easy, as they use static http classes, which passes the Host-Type via constructors |
@rarkins WDYT? |
I really don't know, sorry. Haven't written or used this section of code |
@JamieMagee WDYT? |
This looks reasonable, though from your description it might just push the issue to when updates are fetched? |
🤷♂️ not sure about a better solution |
What is preventing this PR from being merged? |
🎉 This PR is included in version 35.55.2 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Changes
sub-module
manager and fallback togit-refs
git-refs
host-type.Context
Documentation (please check one with an [x])
How I've tested my work (please tick one)
I have verified these changes via: