-
Notifications
You must be signed in to change notification settings - Fork 8
Remove AUTHENTICATION_REQUIRED error code #40
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
Conversation
|
@maxl2287 |
Hey @eric-murray, I am surprised that this occures when there is no new line visible in the file. I am workign with IntelliJ so I do not experienced this behaviour before. Could it be an issue regarding the line-ending in Windows ? CRLF? Idk. 🤷🏻♂️ |
|
@maxl2287 In the absence of line endings being enforced across the project (using a See https://docs.github.com/en/get-started/git-basics/configuring-git-to-handle-line-endings |
I am okay with removing this new-line-at-eof. |
|
Raised #42 |
What type of PR is this?
What this PR does / why we need it:
Commonalities pre-release r3.1 removes the
AUTHENTICATION_REQUIREDerror code, leavingUNAUTHENTICATEDas the only valid error code when the HTTP status code is 401. This PR removes all instances ofAUTHENTICATION_REQUIRED.Which issue(s) this PR fixes:
Fixes #39
Special notes for reviewers:
This is not a breaking change
Changelog input
Additional documentation
None