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

[8.0.x] JettyServerWrapper logs warnings when creating temp dirs on Windows #1841

Closed
mherwege opened this issue Mar 6, 2023 · 1 comment
Closed

[8.0.x] JettyServerWrapper logs warnings when creating temp dirs on Windows #1841

mherwege opened this issue Mar 6, 2023 · 1 comment
Assignees
@grgrzybek
Labels
type: bug Bug
Milestone
8.0.18

Comments

@mherwege
Copy link

mherwege commented Mar 6, 2023

See openhab/openhab-core#3425

I am getting the following JettyServerWrapper warning message in my log:

13:53:06.619 [WARN ] [ce.jetty.internal.JettyServerWrapper] - Can't create temporary directory for OsgiContextModel{HS,id=OCM-17,name='context:1470141928',path='/',bundle=org.apache.felix.webconsole,context=WebContainerContextWrapper{bundle=org.apache.felix.webconsole_4.7.2 [50],contextId='context:1470141928',delegate=org.apache.felix.webconsole.internal.servlet.OsgiManagerHttpContext@57a095e8}}: C:\Users\MHERWEGE\AppData\Local\Temp\ROOT\context:1470141928

It looks like the temp filename is not cleaned for illegal characters on Windows when attempting to create a new temp file.
@grgrzybek grgrzybek self-assigned this Mar 6, 2023
@grgrzybek grgrzybek added the type: bug Bug label Mar 6, 2023
@grgrzybek grgrzybek added this to the 8.0.18 milestone Mar 6, 2023
@grgrzybek
Copy link
Member

Good point - I don't work with Windows, but I can fix this soon.
Thanks for the issue!

@grgrzybek grgrzybek changed the title JettyServerWrapper logs warnings when creating temp dirs on Windows [8.0.x] JettyServerWrapper logs warnings when creating temp dirs on Windows Mar 23, 2023
@grgrzybek grgrzybek mentioned this issue Mar 23, 2023
Closed
grgrzybek added a commit that referenced this issue Mar 23, 2023
@grgrzybek
[Fixes #1841] Fix temporary directory location name on Windows
4c9992c
@Mark-VG Mark-VG mentioned this issue Apr 18, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@grgrzybek grgrzybek

Labels
type: bug Bug
Projects
None yet
Milestone
8.0.18
Development

No branches or pull requests
2 participants
@grgrzybek @mherwege