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

WX-1385 Reject blob URLs with external SAS tokens as unparsable #7347

Merged
merged 5 commits into from
Jan 3, 2024

Merge branch 'develop' into aen_wx_1385

a9a6aeb
Select commit
Loading
Failed to load commit list.
Merged

WX-1385 Reject blob URLs with external SAS tokens as unparsable #7347

Merge branch 'develop' into aen_wx_1385
a9a6aeb
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Dec 20, 2023 in 0s

75.00% of diff hit (target 50.00%)

View this Pull Request on Codecov

75.00% of diff hit (target 50.00%)

Annotations

Check warning on line 65 in filesystems/blob/src/main/scala/cromwell/filesystems/blob/BlobPathBuilder.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

filesystems/blob/src/main/scala/cromwell/filesystems/blob/BlobPathBuilder.scala#L65

Added line #L65 was not covered by tests

Check warning on line 67 in filesystems/blob/src/main/scala/cromwell/filesystems/blob/BlobPathBuilder.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

filesystems/blob/src/main/scala/cromwell/filesystems/blob/BlobPathBuilder.scala#L67

Added line #L67 was not covered by tests

Check warning on line 69 in filesystems/blob/src/main/scala/cromwell/filesystems/blob/BlobPathBuilder.scala

See this annotation in the file changed.

@codecov codecov / codecov/patch

filesystems/blob/src/main/scala/cromwell/filesystems/blob/BlobPathBuilder.scala#L69

Added line #L69 was not covered by tests