Skip to content
This repository was archived by the owner on Jan 10, 2023. It is now read-only.

Add unset of basic auth header in web app config#36

Merged
bflad merged 1 commit intobflad:masterfrom
afklm:no-auth
Oct 6, 2014
Merged

Add unset of basic auth header in web app config#36
bflad merged 1 commit intobflad:masterfrom
afklm:no-auth

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 27, 2014

Resolves possible issues with Stash 2.1+ and XSRF tokens.
See: https://confluence.atlassian.com/display/STASHKB/XSRF+Security+Token+Missing

@ghost
Copy link
Copy Markdown
Author

ghost commented Aug 27, 2014

May I suggest you create a develop branch into which PRs can be accepted? Or at least mention in your CONTRIBUTING.MD file that you accept PRs into master? 😄

@bflad
Copy link
Copy Markdown
Owner

bflad commented Oct 6, 2014

Interesting! I don't think could hurt, so going to pull in. Thanks.

bflad added a commit that referenced this pull request Oct 6, 2014
Add unset of basic auth header in web app config
@bflad bflad merged commit b74d872 into bflad:master Oct 6, 2014
@linc01n
Copy link
Copy Markdown
Collaborator

linc01n commented Nov 4, 2014

My jenkins build notification will fail if we remove the auth header. Even the default backup client fail.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants