Skip to content

fix issue with double encoding of mqtt presigned urls - #750

Open
daphid4 wants to merge 1 commit into
hivemq:masterfrom
Interaction-Gaming:fix/aws-presigned-url-double-encoding-remote
Open

daphid4 wants to merge 1 commit into
hivemq:masterfrom
Interaction-Gaming:fix/aws-presigned-url-double-encoding-remote

Conversation

@daphid4

@daphid4 daphid4 commented Jan 7, 2026

Copy link
Copy Markdown

Description

This fixes a bug found when using AWS presigned urls - the consensus is that the url was being double encoded and therefore always resulting in a 403

Related Issue

#643
https://community.hivemq.com/t/connect-mqtt-broker-with-pre-signed-url/751/8

Type of Change

  • 📚 Examples / docs / tutorials / dependencies update
  • 🔧 Bug fix (non-breaking change which fixes an issue)
  • 🥂 Improvement (non-breaking change which improves an existing feature)
  • 🚀 New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to change)
  • 🔐 Security fix

Checklist

  • I've written tests (if applicable) for all new methods and classes that I created.
  • I've added documentation as necessary so users can easily use and understand this feature/fix.

@cla-bot

cla-bot Bot commented Jan 7, 2026

Copy link
Copy Markdown

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have the users @daphid4 on file. In order for us to review and merge your code, please sign our Contributor License Agreement to get yourself added. You'll find the CLA and more information here: https://github.com/hivemq/hivemq-community/blob/master/CONTRIBUTING.adoc#contributor-license-agreement

@daphid4

daphid4 commented Jan 7, 2026

Copy link
Copy Markdown
Author

it looks like I'm not able to sign the cla
image

@lsiepel

lsiepel commented Apr 1, 2026

Copy link
Copy Markdown

Gentle ping @Donnerbart could you have a look?

This branch has not been deployed

No deployments
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.

2 participants