Skip to content

Conversation

@pawelstepien-da
Copy link
Contributor

@pawelstepien-da pawelstepien-da commented Nov 27, 2025

Added server object in config and removed url-related properties from kernelInfo.
Replaced all hardcoded occurrences of host, port and dapp/user api paths.
It's a breaking change - config has to be adjusted with server and at least properties:
-host
-port
-tls

Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
@pawelstepien-da pawelstepien-da linked an issue Nov 27, 2025 that may be closed by this pull request
@pawelstepien-da pawelstepien-da marked this pull request as ready for review November 27, 2025 13:01
@pawelstepien-da pawelstepien-da requested review from a team as code owners November 27, 2025 13:01
@pawelstepien-da pawelstepien-da marked this pull request as draft November 27, 2025 13:04
Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
…pp-ledger-url

Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
Signed-off-by: Pawel Stepien <pawel.stepien@digitalasset.com>
@pawelstepien-da pawelstepien-da marked this pull request as ready for review November 28, 2025 14:25
@pawelstepien-da pawelstepien-da merged commit 49b5d50 into main Dec 3, 2025
22 of 23 checks passed
@pawelstepien-da pawelstepien-da deleted the pawel/dapp-ledger-url branch December 3, 2025 09:15
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.

Derive server host and port from config

3 participants