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.

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

BugFix: Preserve ports in URI #16

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

BugFix: Preserve ports in URI #16

wants to merge 1 commit into from

Conversation

lszinv
Copy link
Collaborator

@lszinv lszinv commented Jan 17, 2025

Bugfix to preserve ports in the URI field.

Before the fix:
Before

After the fix:
After

Will need to perform some verification with an existing server prior to merging. Due to some issues, I cannot test this scenario with our Tableau Server setup.

Copy link

Code Coverage Summary

Summary
  Generated on: 01/17/2025 - 20:33:24
  Parser: MultiReport (2x OpenCover)
  Assemblies: 2
  Classes: 52
  Files: 63
  Line coverage: 77.2%
  Covered lines: 1173
  Uncovered lines: 346
  Coverable lines: 1519
  Total lines: 5323
  Branch coverage: 56.8% (244 of 429)
  Covered branches: 244
  Total branches: 429
  Method coverage: 84.8% (229 of 270)
  Full method coverage: 45.5% (123 of 270)
  Covered methods: 229
  Fully covered methods: 123

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant