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

Bump up resteasy-client to 3.0.23.Final #772

Merged
merged 3 commits into from
Apr 28, 2018
Merged

Bump up resteasy-client to 3.0.23.Final #772

merged 3 commits into from
Apr 28, 2018

Conversation

takanamito
Copy link
Contributor

Bump up resteasy-client which includes this bug fix.
resteasy/resteasy#1169

This issue is referred to #756

@muga muga self-requested a review April 26, 2018 06:21
@muga muga removed their request for review April 26, 2018 06:51
@muga
Copy link
Member

muga commented Apr 26, 2018

a lot of changes are included..
resteasy/resteasy@3.0.13.Final...3.0.23.Final

@takanamito
Copy link
Contributor Author

@muga However, reported bug is fixed by 3.0.23.Final.
Should I send some PRs which includes updating resteasy version little by little ?

@muga
Copy link
Member

muga commented Apr 28, 2018

@takanamito Thank you for upgrading! It looks good to me. Just it was hard to take a look a lot of changes..

@muga muga merged commit 7224aed into treasure-data:master Apr 28, 2018
@muga muga changed the title [wip] Bump up resteasy-client to 3.0.23.Final Bump up resteasy-client to 3.0.23.Final Apr 28, 2018
@takanamito
Copy link
Contributor Author

Sorry, I misunderstood. Thank you for approving my commit 😄

@muga muga added this to the v0.9.25 milestone May 16, 2018
@muga muga modified the milestones: v0.9.25, v0.9.26 May 25, 2018
@muga
Copy link
Member

muga commented May 25, 2018

@takanamito Sorry, I will slide this PR from v0.9.25 to v0.9.26. After upgrading resteasy version, Digdag server show the following warning messages in access log. I believe that the warning is not critical but, just in case, I will take a look and fix it.

2018-05-24 21:28:37 -0700 [WARN] (XNIO-1 task-9): RESTEASY002142: Multiple resource methods match request "GET /assets/app.css". Selecting one. Matching methods: [public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getApplication(java.lang.String) throws java.io.IOException, public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getAssets(java.lang.String) throws java.io.IOException]
2018-05-24 21:28:37 -0700 [WARN] (XNIO-1 task-8): RESTEASY002142: Multiple resource methods match request "GET /assets/bootstrap.css". Selecting one. Matching methods: [public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getApplication(java.lang.String) throws java.io.IOException, public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getAssets(java.lang.String) throws java.io.IOException]
2018-05-24 21:28:37 -0700 [WARN] (XNIO-1 task-7): RESTEASY002142: Multiple resource methods match request "GET /assets/app.js". Selecting one. Matching methods: [public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getApplication(java.lang.String) throws java.io.IOException, public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getAssets(java.lang.String) throws java.io.IOException]
2018-05-24 21:28:37 -0700 [WARN] (XNIO-1 task-10): RESTEASY002142: Multiple resource methods match request "GET /assets/bootstrap.js". Selecting one. Matching methods: [public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getApplication(java.lang.String) throws java.io.IOException, public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getAssets(java.lang.String) throws java.io.IOException]
2018-05-24 21:28:37 -0700 [WARN] (XNIO-1 task-15): RESTEASY002142: Multiple resource methods match request "GET /assets/448c34a56d699c29117adc64c43affeb.woff2". Selecting one. Matching methods: [public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getApplication(java.lang.String) throws java.io.IOException, public javax.ws.rs.core.Response io.digdag.server.rs.UiResource.getAssets(java.lang.String) throws java.io.IOException]

@muga
Copy link
Member

muga commented May 25, 2018

follow-up PR is #779

@takanamito
Copy link
Contributor Author

Thank you for letting me know.
I wait for fixing it.

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.

None yet

2 participants