Go Windows Azure Client Library

Merge lp://staging/~axwalk/gwacl/management-api-307-redirects into lp://staging/gwacl

  1. management-api-307-redirects
  2. Merge into trunk
Proposed by Andrew Wilkins
Status: Merged
Approved by: Andrew Wilkins
Approved revision: 232
Merged at revision: 231
Proposed branch: lp://staging/~axwalk/gwacl/management-api-307-redirects
Merge into: lp://staging/gwacl
Diff against target: 185 lines (+104/-3)
3 files modified
x509dispatcher.go (+35/-1)
x509dispatcher_test.go (+56/-2)
x509session.go (+13/-0)
To merge this branch: bzr merge lp://staging/~axwalk/gwacl/management-api-307-redirects
Reviewer Review Type Date Requested Status
Ian Booth Approve
Review via email: mp+204618@code.staging.launchpad.net

Commit message

Add handling of temporary redirects in the management API.

Description of the change

Add handling of temporary redirects in the management API.

To post a comment you must log in.
Revision history for this message
Ian Booth (wallyworld) wrote :

Looks good. Did you test it live? I think that should be done before landing.

review: Approve
Revision history for this message
Andrew Wilkins (axwalk) wrote :

I tested bootstrap and destroy-env a bunch of times. I'll run some deployment tests before landing.

Revision history for this message
Andrew Wilkins (axwalk) wrote :

Seems there's still a problem.

lp://staging/~axwalk/gwacl/management-api-307-redirects updated
232. By Andrew Wilkins

Also check redirect status code for POST/DELETE, etc.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: