Skip to content

Fix: Add missing slash in CyberArk request URLs#1046

Merged
greenbonebot merged 1 commit intomainfrom
fix-cyberark-path
Feb 6, 2026
Merged

Fix: Add missing slash in CyberArk request URLs#1046
greenbonebot merged 1 commit intomainfrom
fix-cyberark-path

Conversation

@timopollmeier
Copy link
Member

What

In cyberark_send_request a slash is added to the URL, separating the hostname and optional port from the path.

Why

This makes the connection work without having to add a slash a the start of the path manually.

References

GEA-1562

@timopollmeier timopollmeier requested review from a team as code owners February 4, 2026 09:22
@greenbonebot greenbonebot enabled auto-merge (rebase) February 4, 2026 09:22
@github-actions
Copy link

github-actions bot commented Feb 4, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA e2f565b.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@mattmundell mattmundell disabled auto-merge February 4, 2026 09:34
Copy link
Contributor

@mattmundell mattmundell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what the callers are, but you could also support both (add the / if it's missing).

In cyberark_send_request a slash is added to the URL, separating the
hostname and optional port from the path.

This makes the connection work without having to add a slash a the start
of the path manually.
@greenbonebot greenbonebot enabled auto-merge (rebase) February 6, 2026 08:31
@greenbonebot greenbonebot merged commit 0053b00 into main Feb 6, 2026
17 checks passed
@greenbonebot greenbonebot deleted the fix-cyberark-path branch February 6, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants