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

add optional HTTP basic authentication #141

Merged
merged 1 commit into from
Jul 27, 2015
Merged

add optional HTTP basic authentication #141

merged 1 commit into from
Jul 27, 2015

Conversation

bacoboy
Copy link
Contributor

@bacoboy bacoboy commented Jun 25, 2015

If your Marathon endpoint is authenticated, Bamboo needs to pass Username/Password.
This simple implementation uses the built-in Go net/http SetBasicAuth() if both the user & password are specified.

@j1n6
Copy link
Contributor

j1n6 commented Jun 26, 2015

Hi, thanks for the PR.
I just need to run a few tests, otherwise this look good.

We are moving offices, please expecting slow responses from me.

On 25 Jun 2015, at 21:59, Steve Hoffman notifications@github.com wrote:

If your Marathon endpoint is authenticated, Bamboo needs to pass Username/Password.
This simple implementation uses the built-in Go net/http SetBasicAuth() if both the user & password are specified.

You can view, comment on, or merge this pull request online at:

#141

Commit Summary

add optional HTTP basic authentication
File Changes

M README.md (2)
M configuration/configuration.go (2)
M configuration/marathon.go (2)
M main/bamboo/bamboo.go (3)
Patch Links:

https://github.com/QubitProducts/bamboo/pull/141.patch
https://github.com/QubitProducts/bamboo/pull/141.diff

Reply to this email directly or view it on GitHub.

@bacoboy
Copy link
Contributor Author

bacoboy commented Jul 27, 2015

Just checking in a month later. How'd the move go?

j1n6 pushed a commit that referenced this pull request Jul 27, 2015
add optional HTTP basic authentication
@j1n6 j1n6 merged commit 2d7f218 into QubitProducts:master Jul 27, 2015
@j1n6
Copy link
Contributor

j1n6 commented Jul 27, 2015

Your PR looks good. The move was fun and new office is really nice!
If you are around central London, I would love to invite you.

Thanks for the PR.

@bacoboy
Copy link
Contributor Author

bacoboy commented Jul 28, 2015

Great. Any idea when a point release can be cut? We have somebody here waiting to use it and would like to use the official version over my fork.

Thanks. And if I'm in London I might just take you up on that. Always looking for a reason to grab a cider!

@j1n6
Copy link
Contributor

j1n6 commented Jul 29, 2015

Just made a release. :)

@bacoboy
Copy link
Contributor Author

bacoboy commented Jul 31, 2015

Sweet. I'll check it out. Thanks!

@brugidou brugidou mentioned this pull request Aug 4, 2015
@j1n6
Copy link
Contributor

j1n6 commented Aug 20, 2015

@bacoboy We are at MesosConf. Let me if you would like to meet up :P

@bacoboy
Copy link
Contributor Author

bacoboy commented Aug 21, 2015

Sure! DM me on Twitter @bacoboy. What's your handle?

@j1n6
Copy link
Contributor

j1n6 commented Aug 21, 2015

My handle is @activars

We can catchup in the event this evening.

On 20 Aug 2015, at 17:49, Steve Hoffman notifications@github.com wrote:

Sure! DM me on Twitter @bacoboy. What's your handle?


Reply to this email directly or view it on GitHub.

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

Successfully merging this pull request may close these issues.

2 participants