Skip to content
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 machine auth by using own user token instead of workspace owner token. #10249

Merged
merged 1 commit into from Jul 3, 2018

Conversation

mshaposhnik
Copy link
Contributor

What does this PR do?

Fixes wrong behavior when in shared workspace all users authorize http-requests with the same token created on workspace startup. Now own user token will be used.

What issues does this PR fix or reference?

#10243

Release Notes

N/A

Docs PR

N/A

@mshaposhnik mshaposhnik added the kind/bug Outline of a bug - must adhere to the bug report template. label Jul 3, 2018
@mshaposhnik mshaposhnik self-assigned this Jul 3, 2018
@mshaposhnik
Copy link
Contributor Author

ci-test

@benoitf benoitf added the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jul 3, 2018
@mshaposhnik mshaposhnik merged commit ed187d7 into master Jul 3, 2018
@mshaposhnik mshaposhnik deleted the fix_user_auth branch July 3, 2018 14:20
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Jul 3, 2018
@benoitf benoitf added this to the 6.8.0 milestone Jul 3, 2018
@codenvy-ci
Copy link

ci-test build report:
Build details
Test report
selenium tests report data
docker image: eclipseche/che-server:10249
https://github.com/orgs/eclipse/teams/eclipse-che-qa please check this report.

hbhargav pushed a commit to hbhargav/che that referenced this pull request Dec 5, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Outline of a bug - must adhere to the bug report template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants