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

feat: add support to inject headers into Admin API calls #172

Merged
merged 1 commit into from
Oct 22, 2018

Conversation

hbagdi
Copy link
Member

@hbagdi hbagdi commented Oct 17, 2018

If Kong's control plane is protected behind Authentication or
Authorization, then Kong Ingress Controller cannot talk to Kong. This
change let's user specify flags to add any header(key:value) that will
be added in every call to Kong's Control Plane a.k.a Admin API.

If Kong's control plane is protected behind Authentication or
Authorization, then Kong Ingress Controller cannot talk to Kong. This
change let's user specify flags to add any header(key:value) that will
be added in every call to Kong's Control Plane a.k.a Admin API.
@hbagdi hbagdi requested a review from fffonion October 17, 2018 23:56
@codecov-io
Copy link

Codecov Report

Merging #172 into master will increase coverage by 0.07%.
The diff coverage is 25%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #172      +/-   ##
==========================================
+ Coverage   15.31%   15.39%   +0.07%     
==========================================
  Files          34       35       +1     
  Lines        2820     2846      +26     
==========================================
+ Hits          432      438       +6     
- Misses       2350     2370      +20     
  Partials       38       38
Impacted Files Coverage Δ
internal/ingress/controller/controller.go 0% <ø> (ø) ⬆️
cli/ingress-controller/main.go 0% <0%> (ø) ⬆️
cli/ingress-controller/header_roundtripper.go 0% <0%> (ø)
cli/ingress-controller/flags.go 80.2% <58.33%> (-3.33%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 267f023...c334fda. Read the comment docs.

Copy link
Contributor

@fffonion fffonion left a comment

Choose a reason for hiding this comment

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

🍸

@hbagdi hbagdi merged commit 6bee57b into master Oct 22, 2018
@hbagdi hbagdi deleted the feat/headers branch October 22, 2018 16:24
hbagdi added a commit that referenced this pull request Oct 23, 2018
If Kong's control plane is protected behind Authentication or
Authorization, then Kong Ingress Controller cannot talk to Kong. This
change let's user specify flags to add any header(key:value) that will
be added in every call to Kong's Control Plane a.k.a Admin API.

From #172
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.

3 participants