Skip to content

Url encoding fix#3026

Merged
hazel-bohon merged 5 commits into
masterfrom
url_encoding_fix
Jun 11, 2026
Merged

Url encoding fix#3026
hazel-bohon merged 5 commits into
masterfrom
url_encoding_fix

Conversation

@hazel-bohon

@hazel-bohon hazel-bohon commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

In MSMQ, Message ID (native message id) can contain \ and other characters which must be url encoded in order to be correctly passed to the ServiceControl. This PR adds url encoding when required to allow.

Reviewer Checklist

  • Components are broken down into sensible and maintainable sub-components.
  • Styles are scoped to the component using it. If multiple components need to share CSS, then a .css file is created containing the shared CSS and imported into component scoped style sections.
  • Naming is consistent with existing code, and adequately describes the component or function being introduced
  • Only functions utilizing Vue state or lifecycle hooks are named as composables (i.e. starting with 'use');
  • No module-level state is being introduced. If so, request the PR author to move the state to the corresponding Pinia store.

@hazel-bohon hazel-bohon added Type: Bug Type: Bug javascript Pull requests that update Javascript code labels Jun 9, 2026
@hazel-bohon hazel-bohon requested a review from cquirosj June 9, 2026 15:37
Comment thread src/Frontend/test/preconditions/recoverability.spec.ts Outdated
Comment thread src/Frontend/test/preconditions/recoverability.ts
Comment thread src/Frontend/src/stores/MessageStore.spec.ts
Comment thread src/Frontend/src/router/routeLinks.ts
Comment thread src/Frontend/src/stores/MessageStore.ts
@hazel-bohon hazel-bohon requested a review from cquirosj June 10, 2026 19:50
@hazel-bohon hazel-bohon merged commit 781d376 into master Jun 11, 2026
5 checks passed
@hazel-bohon hazel-bohon deleted the url_encoding_fix branch June 11, 2026 12:00
@DavidBoike DavidBoike modified the milestone: 2.7.2 Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

javascript Pull requests that update Javascript code Type: Bug Type: Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

An error occurred while trying to load the message.

3 participants