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

Resteasy-reactive Unit endpoint reported as "Run time writers required" #42159

Closed
StianHaSu opened this issue Jul 26, 2024 · 2 comments · Fixed by #42195
Closed

Resteasy-reactive Unit endpoint reported as "Run time writers required" #42159

StianHaSu opened this issue Jul 26, 2024 · 2 comments · Fixed by #42195
Labels
Milestone

Comments

@StianHaSu
Copy link

StianHaSu commented Jul 26, 2024

Describe the bug

When returning a Uni with a Unit (using Kotlin), the Dev UI will report a 0/100 for the Writer, with the message: "Run time writers required". This was a bug previously for Uni with a Void as well, as reported in issue: #27338.

Expected behavior

The same Writer score as when returning a Uni with Void

Actual behavior

A score of 0/100, with message: Run time writers required

How to Reproduce?

This repo has a minimal recreation of the bug: https://github.com/StianHaSu/quarkus-Uni-Unit-bug.

  1. Run in dev mode
  2. See endpoint scores in Dev UI

Output of uname -a or ver

No response

Output of java -version

No response

Quarkus version or git rev

No response

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

No response

Additional information

No response

@StianHaSu StianHaSu added the kind/bug Something isn't working label Jul 26, 2024
Copy link

quarkus-bot bot commented Jul 26, 2024

/cc @FroMage (resteasy-reactive), @geoand (kotlin,resteasy-reactive), @stuartwdouglas (resteasy-reactive)

@geoand
Copy link
Contributor

geoand commented Jul 29, 2024

Thanks for spotting this.

#42195 fixes the problem

geoand added a commit that referenced this issue Jul 29, 2024
Treat Kotlin's Unit as void for the Quarkus REST scoring system
@quarkus-bot quarkus-bot bot added this to the 3.14 - main milestone Jul 29, 2024
holly-cummins pushed a commit to holly-cummins/quarkus that referenced this issue Jul 31, 2024
@gsmet gsmet modified the milestones: 3.14 - main, 3.13.1 Aug 1, 2024
gsmet pushed a commit to gsmet/quarkus that referenced this issue Aug 1, 2024
@gsmet gsmet modified the milestones: 3.13.1, 3.8.6 Aug 14, 2024
gsmet pushed a commit to gsmet/quarkus that referenced this issue Aug 14, 2024
danielsoro pushed a commit to danielsoro/quarkus that referenced this issue Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants