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

quarkus.http.cors.exposed-headers not working in POST #34131

Closed
bumbel42so opened this issue Jun 19, 2023 · 2 comments
Closed

quarkus.http.cors.exposed-headers not working in POST #34131

bumbel42so opened this issue Jun 19, 2023 · 2 comments
Labels
area/security env/windows Impacts Windows machines kind/bug Something isn't working triage/duplicate This issue or pull request already exists

Comments

@bumbel42so
Copy link

Describe the bug

I set quarkus.http.cors.exposed-headers but since Quarkus 3.x they only provided in OPTIONS request, not in GET, POST, PUT request. I need this in Angular Frontend to access custom headers.

Expected behavior

I want to get access-control-expose-headers at GET, POST, PUT Request, like in Quarkus 2.16

image

Actual behavior

GET
image

OPTIONS
image
(i also need to add Access-Control-Request-Headers Request Header to get access-control-expose-headers Response Header

How to Reproduce?

  1. application.properties
quarkus.http.cors=true
quarkus.http.cors.origins=*
quarkus.http.cors.exposed-headers=x-custom
  1. send request
  2. see response headers

Output of uname -a or ver

Microsoft Windows [Version 10.0.19045.3086]

Output of java -version

openjdk version "17.0.7" 2023-04-18 OpenJDK Runtime Environment Temurin-17.0.7+7 (build 17.0.7+7) OpenJDK 64-Bit Server VM Temurin-17.0.7+7 (build 17.0.7+7, mixed mode, sharing)

GraalVM version (if different from Java)

No response

Quarkus version or git rev

3.1.2

Build tool (ie. output of mvnw --version or gradlew --version)

Apache Maven 3.8.8

Additional information

No response

@bumbel42so bumbel42so added the kind/bug Something isn't working label Jun 19, 2023
@quarkus-bot quarkus-bot bot added env/windows Impacts Windows machines triage/needs-triage labels Jun 19, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Jun 19, 2023

/cc @pedroigor (bearer-token), @sberyozkin (bearer-token,jwt,security)

@sberyozkin
Copy link
Member

@bumbel42so Thanks for the report, it was fixed with #33923. Sorry about it, closing now as a Duplicate

@sberyozkin sberyozkin added the triage/duplicate This issue or pull request already exists label Jun 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/security env/windows Impacts Windows machines kind/bug Something isn't working triage/duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants