fix: #691 - rstrip the unnecessary rightmost '/' in the confluence url if exists

[danreic]

Description

Fixes: #691

Changes

• added an rstrip to the build_url parameter to remove the unnecessary '' at the confluence path if exists

Testing

• Integration tests passed
• Manual checks performed: added '' to right of the confluence url and reproduced the issue described, then tested with the fix and saw issue solved

Checklist

• Code follows project style guidelines (linting passes).
• Tests added/updated for changes.
• All tests pass locally.
• Documentation updated (if needed).

[SharkyND]

Hey @danreic , thanks for your contribution! Since the original project is no longer actively maintained, we encourage you to move this PR to our active fork.

I have a few friends and colleagues helping maintain this fork, and we urge everyone in the community who can't use the Atlassian cloud offering MCP (https://www.atlassian.com/blog/announcements/remote-mcp-server) to help us maintain this version.

For our version mcp-atlassian we already have or are about to add:

• Dynamic Headers that works with SSE/HTTP-Stream - Done
• Dynamic and scalable Helm chart - Done (we are using it in production at scale)
• Adding Bitbucket tools as a client too - In Progress (PR #7)

We have a few more ideas and would love if you can add them to the discussions there. Also would love if you can try it out and help build this with us!

Thanks!

[danreic]

@SharkyND can you give me access to it?
seems like I dont have permissions to create a PR or push a new branch

[SharkyND]

Hi @danreic usually you would:

• fork the repo to your personal
• create a branch on your fork
• make changes and commit
• github will let you create a PR against the original main.

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork

Let me know if there is something else you had in mind.

Thanks.

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity for 14 days. It will be closed if no further activity occurs. Please leave a comment or remove the 'stale' label if you believe this PR is still relevant. Thank you for your contributions!