Skip to content

Files entitlements need to be case sensitive on Windows #127047

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
Closed
ldematte opened this issue Apr 18, 2025 · 1 comment · Fixed by #126990
Closed

Files entitlements need to be case sensitive on Windows #127047

ldematte opened this issue Apr 18, 2025 · 1 comment · Fixed by #126990
Labels
>bug :Core/Infra/Entitlements Entitlements infrastructure Team:Core/Infra Meta label for core/infra team v8.18.0 v9.0.0

Comments

@ldematte
Copy link
Contributor

Elasticsearch Version

8.18.0, 9.0.0

Installed Plugins

No response

Java Version

bundled

OS Version

Windows

Problem Description

Files entitlements are checked in a case sensitive way. On Windows, this can lead to deny an entitlement (NotEntitlementException) for paths that should be valid.

Steps to Reproduce

The easiest way to reproduce this is to use a lowercase drive letter, e.g. switch to cd c:\path\to\elasticsearch\bin and then launch elasticsearch.bat. ES fails to start, as it checks entitlements for C:\... against c:\...

Logs (if relevant)

No response
@ldematte ldematte added >bug needs:triage Requires assignment of a team area label v9.0.0 v8.18.0 :Core/Infra/Entitlements Entitlements infrastructure and removed needs:triage Requires assignment of a team area label labels Apr 18, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Apr 18, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@ldematte ldematte mentioned this issue Apr 18, 2025
Merged
@ldematte ldematte changed the title Files entitlements are case sensitive on Windows Files entitlements need to be case sensitive on Windows Apr 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>bug :Core/Infra/Entitlements Entitlements infrastructure Team:Core/Infra Meta label for core/infra team v8.18.0 v9.0.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Entitlements] Fix: consider case sensitiveness differences ldematte/elasticsearch
2 participants
@ldematte @elasticsearchmachine