Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix: remove trailing slash in host url #130

Merged
merged 2 commits into from
Feb 14, 2025
Merged

Fix: remove trailing slash in host url #130

merged 2 commits into from
Feb 14, 2025

Conversation

kalisp
Copy link
Member

@kalisp kalisp commented Feb 14, 2025

Changelog Description

Removes trailing issue in Deadline server url which caused issue in retrieval of api/jobs in Submit Image Publishing job to Deadline

Additional review information

Testing notes:

  1. set Deadline server url with trailing slash
    image
  2. publish via DL

@kalisp kalisp added type: bug Something isn't working community Issues and PRs coming from the community members labels Feb 14, 2025
@kalisp kalisp requested review from iLLiCiTiT and MilaKudr February 14, 2025 12:24
@kalisp kalisp self-assigned this Feb 14, 2025
@kalisp kalisp merged commit 2b7f271 into develop Feb 14, 2025
1 check passed
@kalisp kalisp deleted the bugfix/sanitize_host_url branch February 14, 2025 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iLLiCiTiT iLLiCiTiT iLLiCiTiT approved these changes

@MilaKudr MilaKudr Awaiting requested review from MilaKudr

Assignees

@kalisp kalisp

Labels
community Issues and PRs coming from the community members type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kalisp @iLLiCiTiT