UiPath / uipathcli

91%

main: 91%

LAST BUILD ON BRANCH feature/uri-builder

branch: feature/uri-builder

CHANGE BRANCH

x

Reset

• feature/uri-builder
• feature/add-package-documentation
• feature/analyze-feed-auth
• feature/api-package
• feature/avoid-escape-equal-sign
• feature/clear-command
• feature/code-coverage
• feature/collect-coverage-all-os
• feature/command-builder
• feature/command-builder-refactor
• feature/command-errors
• feature/config-command-tenant-list
• feature/configurable-attempts-and-timeout
• feature/configure-tenant-list
• feature/content-length-input
• feature/custom-headers-plugins
• feature/custom-operation-name
• feature/custom-operation-names
• feature/dataservice
• feature/digitize-command-default-project
• feature/download-dotnet-core-runtime
• feature/extend-plugin-parameters
• feature/folder-name
• feature/github-actions-upgrade
• feature/go-1.24
• feature/go1.22.2
• feature/go122
• feature/governance-file
• feature/http-proxy-support
• feature/identity-client
• feature/identity-uri
• feature/improve-command-documentation
• feature/improve-config-set-command
• feature/improve-studio-project-compatibility-error-message
• feature/library-feed-auth
• feature/login-confidential-app
• feature/network-package
• feature/orchestrator-v17
• feature/pack-output-type-validation
• feature/pat-documentation
• feature/plugins-offline
• feature/project-builder
• feature/project-pack
• feature/publish-create-release
• feature/readme-example-generative-extractor
• feature/readme-examples
• feature/readme-pat
• feature/refresh-token-support
• feature/rename-service-version
• feature/restore-command
• feature/security-policy
• feature/split-plugin-tests
• feature/split-tests
• feature/split-tests-and-commands
• feature/static-linking
• feature/studio
• feature/studio-folder-name
• feature/studio-optional-parameters
• feature/studio-package-publish
• feature/studio-publish
• feature/studio-publish-command
• feature/test-on-windows-macos
• feature/test-parallel
• feature/test-run-command
• feature/test-run-folder-id
• feature/test-run-junit-robotlogs
• feature/test-run-parallel
• feature/test-run-readme
• feature/test-sequence
• feature/update-codeowners
• feature/update-dependencies
• feature/update-dependencies-12-2024
• feature/update-deps-latest-minor
• feature/update-examples
• feature/update-latest-definitions-01-2024
• feature/update-latest-definitions-08-2024
• feature/update-latest-definitions-12-2024
• feature/upgrade-dependencies-v2
• feature/upgrade-deps
• feature/upgrade-go-1.23
• feature/upgrade-golangci-v2
• feature/upgrade-openapi-specifications
• feature/upgrade-openapi-specs
• feature/user-agent
• feature/utils-split-up
• feature/version-command
• feature/version-flag
• fix/library-auth-identity-url
• fix/short-help-option
• fix/show-example-usage-nested-object
• fix/studio-package-progressbar
• fix/uri-builder
• fix/use-testdata-templates
• main
• refs/tags/plugins-v2.0.0
• refs/tags/plugins.v2.0.0
• refs/tags/plugins/v2.0.0
• refs/tags/plugins_v2.0.0
• refs/tags/v2.0.0-plugin
• refs/tags/v2.0.0-uipcli

Build # 14878614187

Build Type

Pull #191

github

Committed by thschmitt

Commit Message

Move URI formatting logic from plugins into UriBuilder

Each plugin has been doing some custom formatting of the URI and
replacing organization and tenant. Moved all the logic for URI handling
in the API clients which use the `uri_builder.go`.

Changed all the code which uses the baseUri to make use of url.URL
instead of strings which allowed removing the `NewUriBuilder` method
which used to convert the baseUri string.

Noticed that the UriBuilder does not espace all URI parts properly.
Added escaping for path and query string keys.

Added more tests to validate the url formatting to avoid similar issues
like https://github.com/UiPath/uipathcli/issues/189.

Pull Request Pull Request #191: Move URI formatting logic from plugins into UriBuilder

Run Details

160 of 163 new or added lines in 15 files covered. (98.16%)

1 existing line in 1 file now uncovered.

6421 of 7092 relevant lines covered (90.54%)

1.02 hits per line