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

Underscore Header Bad Request #264

Closed
1 of 3 tasks
sleekflowleochoi opened this issue Jun 10, 2022 · 6 comments
Closed
1 of 3 tasks

Underscore Header Bad Request #264

sleekflowleochoi opened this issue Jun 10, 2022 · 6 comments
Assignees
Labels
investigating currently looking into the issue Networking Related to ACA networking

Comments

@sleekflowleochoi
Copy link

sleekflowleochoi commented Jun 10, 2022

Please provide us with the following information:

This issue is a: (mark with an x)

  • bug report -> please search issues before submitting
  • documentation issue or request
  • regression (a behavior that used to work and stopped in a new release)

Issue description

When I request my container app with the header
Sfdc_stack_depth: 1
Then the container app responds "Bad Request" but I confirm the request is not reached my runtime.

When I request without Sfdc_stack_depth, then the request is reached my runtime.

https://stackoverflow.com/questions/47005217/enable-underscores-in-headers-on-azure-ingress

I can't customize the setting in azure container app.

Steps to reproduct

  1. POST request with Sfdc_stack_depth
@ghost ghost added the Needs: triage 🔍 Pending a first pass to read, tag, and assign label Jun 10, 2022
@sleekflowleochoi sleekflowleochoi changed the title Underscore Header Underscore Header Bad Request Jun 10, 2022
@dariagrigoriu dariagrigoriu added investigating currently looking into the issue and removed Needs: triage 🔍 Pending a first pass to read, tag, and assign labels Jun 13, 2022
@andlju
Copy link

andlju commented Jul 22, 2022

I'm also running into this problem and it is currently blocking me from moving to Container Apps. If anyone can find a work-around that would be amazing!

@andlju
Copy link

andlju commented Jul 22, 2022

Update: Something that worked for me was to use DAPR for the HTTP calls instead of the ingress endpoint. Unfortunately that doesn't solve the problem if you need to access the app outside of the container apps environment.

@leoiii12
Copy link

Finally, I placed a azure front door in front of the azure container apps. The front door acts a an apigw to change the underscore header to a normal header, or simply remove the underscroe header if you don't need to read it.

@kendallroden kendallroden added the Networking Related to ACA networking label Nov 2, 2022
@uabarahona
Copy link

uabarahona commented Nov 12, 2022

Same on my case, I'm currently deploying a normal next.js app, this app though is doing some request using the following headers:

__rsc__: 1
__next_router_state_tree__": ["",{"children":["",{}]},null,null,true]

the request is not reaching the runtime, and these headers are needed since not including them produces a different result than including them.


UPDATE: My case has been fixed by the next.js team, they have replaced the headers vercel/next.js#42482, I will look latter why is underscores are commonly not accepted, thanks team azure container apps is looking amazing.

@onionhammer
Copy link

@ahmelsayed this one can probably be closed as well

@ahmelsayed
Copy link
Member

This should work now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigating currently looking into the issue Networking Related to ACA networking
Projects
None yet
Development

No branches or pull requests

9 participants