Skip to content

Comments

feat: add --insecure flag for TLS verification skip#664

Merged
jhkimqd merged 4 commits intomainfrom
jihwan/ulxly-insecure-flag
Aug 6, 2025
Merged

feat: add --insecure flag for TLS verification skip#664
jhkimqd merged 4 commits intomainfrom
jihwan/ulxly-insecure-flag

Conversation

@jhkimqd
Copy link
Contributor

@jhkimqd jhkimqd commented Aug 5, 2025

Description

This PR adds a new --insecure flag in polycli ulxly commands to skip TLS verification to avoid something like:

DEBUG: Claim command output for test 6: 9:57AM FTL Received critical error error="Get \"https://bridge.cardona-24.dev.polygon/bridge?net_id=0&deposit_cnt=177657\": tls: failed to verify certificate: x509: certificate signed by unknown authority"

Reference

Foundry cast also had a similar issue, which was resolved in a later PR.

Related tickets

Testing

Signed-off-by: Ji Hwan <jkim@polygon.technology>
@jhkimqd jhkimqd marked this pull request as draft August 5, 2025 06:30
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
@jhkimqd jhkimqd force-pushed the jihwan/ulxly-insecure-flag branch from 0b92eca to fac5e54 Compare August 5, 2025 08:16
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Copy link
Member

@leovct leovct left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jhkimqd jhkimqd marked this pull request as ready for review August 6, 2025 07:47
@jhkimqd jhkimqd merged commit 347eb59 into main Aug 6, 2025
14 checks passed
@jhkimqd jhkimqd deleted the jihwan/ulxly-insecure-flag branch August 6, 2025 07:48
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