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.

Sign up for GitHub

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

Jump to bottom

REST Client does not encode multi character code points correctly #10245

Closed
stuartwdouglas opened this issue Jun 25, 2020 · 5 comments
Closed

REST Client does not encode multi character code points correctly #10245

stuartwdouglas opened this issue Jun 25, 2020 · 5 comments
Labels
kind/bug Something isn't working
Milestone
1.9.0.CR1

Comments

@stuartwdouglas
Copy link
Member

If you attempt to encode a string that contains code points that don't fit into a single char (e.g. emojis) RESTEasy will fail to encode it correctly.
@stuartwdouglas stuartwdouglas added the kind/bug Something isn't working label Jun 25, 2020
stuartwdouglas added a commit to stuartwdouglas/quarkus that referenced this issue Jun 25, 2020
@stuartwdouglas
REST client encoding test
5c759e3 
Test for quarkusio#10245
@stuartwdouglas stuartwdouglas mentioned this issue Jun 25, 2020
Merged
@jaikiran
Copy link
Member

Hello @stuartwdouglas, I guess this is in context of #10134?

@stuartwdouglas
Copy link
Member Author

Oops, I did not realize there was already an issue for it.

@jaikiran jaikiran mentioned this issue Jun 25, 2020
Closed
@haseemisaac
Copy link

Hi @stuartwdouglas, has this been fixed? If not in the mean time, what would be the way to go around it?

@stuartwdouglas
Copy link
Member Author

It won't be fixed until there is a RESTEasy release that includes my PR and this is integrated into Quarkus.

gastaldi pushed a commit to stuartwdouglas/quarkus that referenced this issue Oct 9, 2020
@stuartwdouglas @gastaldi
REST client encoding test
db1a7c3 
Test for quarkusio#10245
@gastaldi gastaldi added this to the 1.9.0.Final milestone Oct 9, 2020
@gastaldi
Copy link
Contributor

gastaldi commented Oct 9, 2020

Fixed in 1.9.0.CR1 with 6c08bcd

@gastaldi gastaldi closed this as completed Oct 9, 2020
@gastaldi gastaldi modified the milestones: 1.9.0.Final, 1.9.0.CR1 Oct 9, 2020
xumk pushed a commit to xumk/quarkus that referenced this issue Oct 14, 2020
@stuartwdouglas @xumk
REST client encoding test
335e089 
Test for quarkusio#10245
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Something isn't working
Projects
None yet
Milestone
1.9.0.CR1
Development

No branches or pull requests
4 participants
@gastaldi @jaikiran @stuartwdouglas @haseemisaac