Skip to content

fix: handle insecure-tls in all commands#875

Closed
fsamin wants to merge 1 commit intojfrog:dev-deprecated-please-target-master-directlyfrom
fsamin:dev
Closed

fix: handle insecure-tls in all commands#875
fsamin wants to merge 1 commit intojfrog:dev-deprecated-please-target-master-directlyfrom
fsamin:dev

Conversation

@fsamin
Copy link

@fsamin fsamin commented Oct 27, 2020

I wasn't able to run the access-token-create command with the --insecure-tls flag.
So I suggest to handle the insecure-tls in all commands. I confirm that the lower layers of the CLI handle it perfectly.

Signed-off-by: francois samin francois.samin@corp.ovh.com

  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • This pull request is on the dev branch.
  • I used gofmt for formatting the code before submitting the pull request.

Signed-off-by: francois  samin <francois.samin@corp.ovh.com>
@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2020

CLA Assistant Lite bot All contributors have signed the CLA ✍️

@fsamin
Copy link
Author

fsamin commented Oct 27, 2020

I have read the CLA Document and I hereby sign the CLA

@eyalbe4
Copy link
Contributor

eyalbe4 commented Dec 13, 2020

@fsamin,
Thanks for creating this PR.
Just adding the --insecure-tls flag to those commands will not make it work. This is because the implementation does not use it. Unlike the mvn command, which translates this flag into maven equivalent maven options, gradle or the docker commands do not.

@eyalbe4
Copy link
Contributor

eyalbe4 commented Mar 29, 2021

@fsamin,
Please let me know if you'd like to keep this PR open.

@fsamin
Copy link
Author

fsamin commented Mar 31, 2021

Hello,
Yes, please keep this open.
I have to rebase and check all the commands that can support --insecure-tls (i.e targeting Artifactory API) and the others.

@ehl-jf ehl-jf added the stale An outdated issue lacking engagement label Sep 24, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

This pull-request was closed due to 7 days of inactivity after being marked as stale. Feel free to reopen it if it remains relevant.

@github-actions github-actions bot closed this Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale An outdated issue lacking engagement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants