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

Client re-auth failures #66

Open
wallyworld opened this issue Jul 31, 2018 · 5 comments
Open

Client re-auth failures #66

wallyworld opened this issue Jul 31, 2018 · 5 comments
Assignees

Comments

@wallyworld
Copy link
Member

The symptom is that pushes of resources (or even charms) to the staging charmstore fails.

The problem is though to be here https://github.com/go-goose/goose/blob/v2/client/client.go#L219.
In the case where it has an expired authentication token then performs the first PUT get's an authorization failure re-authorizes with keystone and just tries again, without ever trying to reset the reader containing the request body, as it's already read it once it's at EOF and thus fails on the second attempt.

@mitechie
Copy link
Contributor

mitechie commented Aug 6, 2018

@hmlanigan can you please look at this bug first before the lxd issues for the week?

Thanks

@hmlanigan
Copy link
Member

started looking

@hmlanigan
Copy link
Member

@wallyworld, I agree that the goose client doesn't correctly reset an resp.Body for reuse, but i haven't found the linkage from goose/swift to the charmstore yet?

@hmlanigan
Copy link
Member

the last version of charmstore that used goose/swift is 5.3.1

@mhilton
Copy link
Contributor

mhilton commented Aug 9, 2018

#70 contains a potential fix for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants