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

Added X-Forwarded-Host header #7

Merged
merged 4 commits into from
Jun 21, 2016
Merged

Conversation

domano
Copy link
Contributor

@domano domano commented Jun 21, 2016

No description provided.

@smancke
Copy link
Member

smancke commented Jun 21, 2016

+1

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.539% when pulling 5b58477 on add-X-Fordwarded-Host-Request-Header into 20d3c39 on master.

package util

import (
mockHttp "content-ui-service/mocks/net/http"
Copy link
Member

Choose a reason for hiding this comment

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

travis build is red, because import path is wrong ..
maybe just copy over the mock first and do a clean go:generate later on ..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i have added the generation script we use everywhere else and also a line in interfaces.go to generate the mock. the mock is there now too,.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.06%) to 92.599% when pulling 492afb2 on add-X-Fordwarded-Host-Request-Header into 20d3c39 on master.

@domano domano merged commit 5f00f99 into master Jun 21, 2016
@domano domano deleted the add-X-Fordwarded-Host-Request-Header branch June 21, 2016 16:26
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.

None yet

3 participants