Skip to content

Conversation

@rjmateus
Copy link
Member

What does this PR change?

HUB XML-RPC API call should be done through apache and using HTTPS, as we have for saline, for example.
This allows the HUB API to use SSL and avoids opening one more port.

Depends on this PR: uyuni-project/uyuni#10395

Needs documentation adjustments.

Codespace

Check if you already have a running container clicking on Running CodeSpace

Create CodeSpace About billing for Github Codespaces CodeSpace Billing Summary CodeSpace Limit

Test coverage

  • No tests: add explanation

  • No tests: already covered

  • Unit tests were added

  • DONE

Links

Issue(s): #

  • DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Before you merge

Check How to branch and merge properly!

@rjmateus rjmateus force-pushed the uyuni_hub_xml_rpc_through_apache branch from de94fee to fc38a54 Compare June 20, 2025 11:54
Signed-off-by: Ricardo Mateus <rmateus@suse.com>
@rjmateus rjmateus force-pushed the uyuni_hub_xml_rpc_through_apache branch from fc38a54 to 08fa5c6 Compare June 20, 2025 12:00
@sonarqubecloud
Copy link

@rjmateus rjmateus marked this pull request as draft July 7, 2025 11:51
@cbosdo
Copy link
Contributor

cbosdo commented Aug 26, 2025

HUB XML-RPC API call should be done through apache and using HTTPS, as we have for saline, for example. This allows the HUB API to use SSL and avoids opening one more port.

Could this affect existing deployments? I suppose that we cannot remove the hub port without breaking something

@rjmateus
Copy link
Member Author

rjmateus commented Sep 2, 2025

@cbosdo the PR is not finished, and it will probably need to make some "migration" step. However, keep the port exposed during a major version is not a problem, and we can remove it during the upgrade/migration process.

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.

3 participants