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 ObjectStorage expired token (v3) #14175

Merged
merged 2 commits into from
Mar 5, 2019
Merged

Fix ObjectStorage expired token (v3) #14175

merged 2 commits into from
Mar 5, 2019

Conversation

skydiablo
Copy link
Contributor

ObjectStorage Auth-Service (in explizit v3) will recheck the cached-token and will end in an "token expired exception".

Auth-Service (in explizit v3) will recheck the cached-token and will end in an "token expired exception".
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I currently do not have a swift setup to check. But it makes sense.

@rullzer rullzer added bug 3. to review Waiting for reviews labels Feb 13, 2019
@rullzer rullzer added this to the Nextcloud 16 milestone Feb 13, 2019
Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code makes sense 👍

@MorrisJobke
Copy link
Member

This fixes #14155

@MorrisJobke MorrisJobke mentioned this pull request Mar 4, 2019
45 tasks
@MorrisJobke MorrisJobke merged commit debd32b into nextcloud:master Mar 5, 2019
@welcome
Copy link

welcome bot commented Mar 5, 2019

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22
Most developers hang out on IRC. So join #nextcloud-dev on Freenode for a chat!

@MorrisJobke MorrisJobke mentioned this pull request Mar 6, 2019
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants