Tar API PR accidentally reverted another PR

[cremor]

PR #67883 by @carlossanlop created the method ArchivingUtils.AttemptSetLastWriteTime by moving its code out of the previously only call site. But it seems like something went wrong there, because the moved code wasn't exactly the same. The code in the new method AttemptSetLastWriteTime only catches UnauthorizedAccessException while the old code directly in ZipFileExtensions.ExtractToFile did catch all exceptions. So this reverted PR #63912 by @EatonZ.

Added code: https://github.com/dotnet/runtime/pull/67883/files#diff-23247c9ec493be8075d059e176db7ef75c268642f1efda7a01a438cc45513636
Removed code: https://github.com/dotnet/runtime/pull/67883/files#diff-9ee8cace4e5c7b1b15c3242da2bc0ed401bd870b9834263a1693ad914183b4b4 (load the diff for ZipFileExtensions.ZipArchiveEntry.Extract.cs)

I assume this was an accidental change, so this should be fixed so that all exceptions are cought again.

[dotnet-issue-labeler]

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

[ghost]

Tagging subscribers to this area: @dotnet/area-system-io-compression
See info in area-owners.md if you want to be subscribed.

Issue Details

---

PR #67883 by @carlossanlop created the method ArchivingUtils.AttemptSetLastWriteTime by moving its code out of the previously only call site. But it seems like something went wrong there, because the moved code wasn't exactly the same. The code in the new method AttemptSetLastWriteTime only catches UnauthorizedAccessException while the old code directly in ZipFileExtensions.ExtractToFile did catch all exceptions. So this reverted PR #63912 by @EatonZ.

Added code: https://github.com/dotnet/runtime/pull/67883/files#diff-23247c9ec493be8075d059e176db7ef75c268642f1efda7a01a438cc45513636
Removed code: https://github.com/dotnet/runtime/pull/67883/files#diff-9ee8cace4e5c7b1b15c3242da2bc0ed401bd870b9834263a1693ad914183b4b4 (load the diff for ZipFileExtensions.ZipArchiveEntry.Extract.cs)

I assume this was an accidental change, so this should be fixed so that all exceptions are cought again.

Author:	cremor
Assignees:	-
Labels:	area-System.IO.Compression, untriaged
Milestone:	-

[ghost]

Tagging subscribers to this area: @dotnet/area-system-io
See info in area-owners.md if you want to be subscribed.

Issue Details

---

PR #67883 by @carlossanlop created the method ArchivingUtils.AttemptSetLastWriteTime by moving its code out of the previously only call site. But it seems like something went wrong there, because the moved code wasn't exactly the same. The code in the new method AttemptSetLastWriteTime only catches UnauthorizedAccessException while the old code directly in ZipFileExtensions.ExtractToFile did catch all exceptions. So this reverted PR #63912 by @EatonZ.

Added code: https://github.com/dotnet/runtime/pull/67883/files#diff-23247c9ec493be8075d059e176db7ef75c268642f1efda7a01a438cc45513636
Removed code: https://github.com/dotnet/runtime/pull/67883/files#diff-9ee8cace4e5c7b1b15c3242da2bc0ed401bd870b9834263a1693ad914183b4b4 (load the diff for ZipFileExtensions.ZipArchiveEntry.Extract.cs)

I assume this was an accidental change, so this should be fixed so that all exceptions are cought again.

Author:	cremor
Assignees:	-
Labels:	area-System.IO, untriaged
Milestone:	-