You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We have started to receive messages from Github regarding deprecation of Basic Auth against API. Looking at the code, it appears that token will never be used even if specified. The code in pullRequests.js appears to only pass the Basic Auth credentials from the global config and never passes the token. The github.js code either way will always override token with Basic Auth (latter comes later in sequence and there is not check that auth object is already set).
Also interested to understand when the Github app is used compared to the other Auth credentials. It looks like Pull Request updates only use Basic/Token credentials. Correct?
The text was updated successfully, but these errors were encountered:
Thank you very much for pointing to it. You are right, it is kind of legacy code, we use there. So there are couple of places, where we call github with basic authentication, mainly for handling comments in PRs.
If you wish you can update the code and switch the calls to token authentication, then we can also get rid of the check in github.js
Hi @nycnewman, Thank you once again for pointing out the depreciation notice.
We have replaced the APIs using Basic Auth credentials with the global config (claassistantio) token. These changes are reflected in our latest release v2.4.6.
We have started to receive messages from Github regarding deprecation of Basic Auth against API. Looking at the code, it appears that token will never be used even if specified. The code in pullRequests.js appears to only pass the Basic Auth credentials from the global config and never passes the token. The github.js code either way will always override token with Basic Auth (latter comes later in sequence and there is not check that auth object is already set).
Also interested to understand when the Github app is used compared to the other Auth credentials. It looks like Pull Request updates only use Basic/Token credentials. Correct?
The text was updated successfully, but these errors were encountered: