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

[8.0] Request Backport for #91980 #106682

Open
AliveDevil opened this issue Aug 20, 2024 · 2 comments
Open

[8.0] Request Backport for #91980 #106682

AliveDevil opened this issue Aug 20, 2024 · 2 comments
Milestone

Comments

@AliveDevil
Copy link

Description

Blocked by .NET 8 failing on setting acl on long paths (#91980)

Please backport this as mentioned here #92460 (comment), or provide out-of-band System.IO.FileSystem.AccessControl.dll as consumable NuGet package.

Reproduction Steps

See description.

Expected behavior

See description.

Actual behavior

See description.

Regression?

No response

Known Workarounds

No response

Configuration

No response

Other information

No response

@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Aug 20, 2024
@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Aug 20, 2024
@vcsjones vcsjones added area-System.IO and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Aug 20, 2024
@adamsitnik adamsitnik added this to the 8.0.x milestone Aug 20, 2024
@adamsitnik adamsitnik added needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration and removed untriaged New issue has not been triaged by the area owner labels Aug 20, 2024
@adamsitnik
Copy link
Member

Blocked by .NET 8 failing on setting acl on long paths

Could you please describe your scenario with a more detail (I need strong business justification to justify backporting)? Thanks!

@AliveDevil
Copy link
Author

AliveDevil commented Aug 20, 2024

Currently in the process of migrating a codebase from .NET framework to .NET 8 due to .NET Framework not handling long paths correctly.

When .NET 8 doesn’t support long paths like this, there’s no point in going forward, as the next possible LTS version is 10 at the end of 2025.

Scenario is a file synchronization tool, which sets Acls on files, based on remote attributes.

As a workaround I’ve used reflection and a lambdaexpression to get access to the Persist(string) overload, to call that directly with the \\?\-prefix.

Business reason: this is a bug in a core LTS library, that is supported for several years. I‘d understand it, if there’s a newer LTS library available, but STS 9 isn’t even released yet, and 8 is the current LTS, while the mentioned PR was merged before 8 got released. This should’ve been fixed by the .NET 8.0.0 release in Nov 2023. I shouldn’t have to resort to reflection to workaround a core bug.

@jeffhandley jeffhandley removed the needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration label Nov 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants