Skip to content

Conversation

@tsegismont
Copy link
Contributor

This simplifies debugging with tools like wireshark.

This simplifies debugging with tools like wireshark.

Signed-off-by: Thomas Segismont <[email protected]>
@tsegismont tsegismont requested review from BillyYccc and vietj June 16, 2021 16:23
@tsegismont
Copy link
Contributor Author

@vietj @BillyYccc tell me if you'd like this for Pg and MySQL too.

@vietj
Copy link
Member

vietj commented Jun 16, 2021

yes please @tsegismont it's a blessing to have such feature

This simplifies debugging with tools like wireshark.

Signed-off-by: Thomas Segismont <[email protected]>
Signed-off-by: Thomas Segismont <[email protected]>
@tsegismont tsegismont removed request for BillyYccc and vietj June 17, 2021 10:03
@tsegismont tsegismont merged commit 1381e48 into eclipse-vertx:master Jun 17, 2021
@tsegismont
Copy link
Contributor Author

I didn't do it for MySQL because some tests don't pass when the flag is present. Not sure why

@tsegismont tsegismont deleted the container_fixed_port branch June 17, 2021 10:08
Rattenkrieg pushed a commit to Rattenkrieg/vertx-sql-client that referenced this pull request Nov 8, 2021
…x#988)

* Allow to configure SQL Server container with a fixed port

This simplifies debugging with tools like wireshark.

Signed-off-by: Thomas Segismont <[email protected]>

* Allow to configure PostgreSQL Server container with a fixed port

This simplifies debugging with tools like wireshark.

Signed-off-by: Thomas Segismont <[email protected]>

* Update documentation

Signed-off-by: Thomas Segismont <[email protected]>
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.

2 participants