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

Add early rejection from RestHandler for unauthorized requests #3418

Merged
merged 120 commits into from
Oct 6, 2023

Match RestRequest implementation of path

f0350f2
Select commit
Loading
Failed to load commit list.
Merged

Add early rejection from RestHandler for unauthorized requests #3418

Match RestRequest implementation of path
f0350f2
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 6, 2023 in 1s

76.82% of diff hit (target 64.59%)

View this Pull Request on Codecov

76.82% of diff hit (target 64.59%)

Annotations

Check warning on line 12 in src/main/java/org/opensearch/security/filter/NettyAttribute.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/NettyAttribute.java#L12

Added line #L12 was not covered by tests

Check warning on line 22 in src/main/java/org/opensearch/security/filter/NettyAttribute.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/NettyAttribute.java#L22

Added line #L22 was not covered by tests

Check warning on line 81 in src/main/java/org/opensearch/security/filter/NettyRequest.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/NettyRequest.java#L81

Added line #L81 was not covered by tests

Check warning on line 98 in src/main/java/org/opensearch/security/filter/NettyRequest.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/NettyRequest.java#L97-L98

Added lines #L97 - L98 were not covered by tests

Check warning on line 36 in src/main/java/org/opensearch/security/filter/NettyRequestChannel.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/NettyRequestChannel.java#L36

Added line #L36 was not covered by tests

Check warning on line 40 in src/main/java/org/opensearch/security/filter/NettyRequestChannel.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/NettyRequestChannel.java#L40

Added line #L40 was not covered by tests

Check warning on line 44 in src/main/java/org/opensearch/security/filter/NettyRequestChannel.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/NettyRequestChannel.java#L44

Added line #L44 was not covered by tests

Check warning on line 157 in src/main/java/org/opensearch/security/filter/SecurityRestFilter.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/SecurityRestFilter.java#L157

Added line #L157 was not covered by tests

Check warning on line 160 in src/main/java/org/opensearch/security/filter/SecurityRestFilter.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/SecurityRestFilter.java#L160

Added line #L160 was not covered by tests

Check warning on line 182 in src/main/java/org/opensearch/security/filter/SecurityRestFilter.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/SecurityRestFilter.java#L182

Added line #L182 was not covered by tests

Check warning on line 3 in src/main/java/org/opensearch/security/filter/SecurityRestUtils.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/filter/SecurityRestUtils.java#L3

Added line #L3 was not covered by tests

Check warning on line 61 in src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java#L61

Added line #L61 was not covered by tests

Check warning on line 94 in src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java#L93-L94

Added lines #L93 - L94 were not covered by tests

Check warning on line 126 in src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java#L123-L126

Added lines #L123 - L126 were not covered by tests

Check warning on line 135 in src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java#L135

Added line #L135 was not covered by tests

Check warning on line 137 in src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

src/main/java/org/opensearch/security/ssl/http/netty/Netty4HttpRequestHeaderVerifier.java#L137

Added line #L137 was not covered by tests