Skip to content

[FIX] Include more ip ranges in WebPush target validation (3.8.x) - #3120

Open
chibenwa wants to merge 4 commits into
apache:3.8.xfrom
chibenwa:bp-jmap-ssrf-3-8-x
Open

[FIX] Include more ip ranges in WebPush target validation (3.8.x)#3120
chibenwa wants to merge 4 commits into
apache:3.8.xfrom
chibenwa:bp-jmap-ssrf-3-8-x

Conversation

@chibenwa

Copy link
Copy Markdown
Contributor

No description provided.

@Arsnael

Arsnael commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

02:03:14,654 [ERROR] /home/jenkins/jenkins-agent/workspace/james_ApacheJames_PR-3120/server/protocols/jmap-rfc-8621/src/test/scala/org/apache/james/jmap/pushsubscription/SafeWebPushClientContract.scala:66: error: not found: type URI
02:03:14,654 [ERROR] assertThatThrownBy(() => Mono.from(testee.push(PushSubscriptionServerURL(new URI("file:///etc/passwd").toURL), PUSH_REQUEST_SAMPLE)).block)

@Arsnael

Arsnael commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Let me fix it

@Arsnael

Arsnael commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Minimal fixup but cleaner IMO, sorry for allowing myself :)

)

* [FIX] Include more ip ranges in WebPush target validation

* fixup! [FIX] Include more ip ranges in WebPush target validation

* fixup! fixup! [FIX] Include more ip ranges in WebPush target validation
@chibenwa
chibenwa force-pushed the bp-jmap-ssrf-3-8-x branch from 5f4b776 to 2dd9c24 Compare August 19, 2026 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants