coderazzi / aws_codeartifact_maven

Support to update AWS CodeArtifact credentials on maven IDEA projects.
MIT License
6 stars 6 forks source link

Bug: maven server id did not show the correct list from the setings.xml #14

Closed Happy-ArthurYang closed 10 months ago

Happy-ArthurYang commented 10 months ago

when I try to work with my mutiple maven repos the list of Maven server Id in the UI did not show all of my server Id in the seetings.xml image Below is my seetiings.xml snapsho, there are 4 servers . image

the expect result is the UI could list all my server in the dropdown list

coderazzi commented 10 months ago

Hi, there is a refresh button for that field on the plugin, so it only parses the settings.xml once, or when needed. If that doesn't work, could you send me your settings. XML file,? Best L.

On Mon, Oct 16, 2023, 02:28 ArthurYang @.***> wrote:

when I try to work with my mutiple maven repos the list of Maven server Id in the UI did not show all of my server Id in the seetings.xml [image: image] https://user-images.githubusercontent.com/113080404/275363801-b3de6d2c-a8d0-4f20-a0a2-d5c3f516a4b7.png Below is my seetiings.xml snapsho, there are 4 servers . [image: image] https://user-images.githubusercontent.com/113080404/275364063-a02d148f-2fdf-4bed-9028-86fe3cac28fc.png

the expect result is the UI could list all my server in the dropdown list

— Reply to this email directly, view it on GitHub https://github.com/coderazzi/aws_codeartifact_maven/issues/14, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAH54IMSZIL6J52R7JPE6MDX7R5T5AVCNFSM6AAAAAA6BL3U5WVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2DIMJSHAZTOMI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Happy-ArthurYang commented 10 months ago

will continue to observe, seems not work every time, especially for my wrapper script scenario. There is a long string to show the script. image

coderazzi commented 10 months ago

Hi, indeed, that looks very bad, Your bat script is now something like:

C:\MyApps\JetBrains....\aws %* --no-verify-ssl

Try having

@C:\MyApps\JetBrains....\aws %* --no-verify-ssl

This way, the line itself won't be included in the output,

Best, L.

On Tue, Oct 17, 2023 at 2:35 AM ArthurYang @.***> wrote:

will continue to observe, seems not work every time, especially for my wrapper script scenario. There is a long string to show the script. [image: image] https://user-images.githubusercontent.com/113080404/275678728-29a0cfa3-11f3-461a-b567-efeebba55b01.png

— Reply to this email directly, view it on GitHub https://github.com/coderazzi/aws_codeartifact_maven/issues/14#issuecomment-1765475589, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAH54IKKX4XR64FVAPP6TUDX7XHGLAVCNFSM6AAAAAA6BL3U5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRVGQ3TKNJYHE . You are receiving this because you commented.Message ID: @.***>

Happy-ArthurYang commented 10 months ago

getTokenForGenie.txt getTokenForTlm.txt getTokenForTlp.txt getTokenForTmc.txt I list all my scripts just change the ext from .cmd to .txt

Happy-ArthurYang commented 10 months ago

this is the UI before click GEnerate Credentials button. image

coderazzi commented 10 months ago

Hi, your scriptgetTokenForGenie.cmd should be:

@aws codeartifact get-authorization-token --domain genie-domain --domain-owner 380723925535 --query authorizationToken --output text --profile genie-ss --no-verify-ssl

instead of

aws codeartifact get-authorization-token --domain genie-domain --domain-owner 380723925535 --query authorizationToken --output text --profile genie-ss --no-verify-ssl

(attention to the first @ character)

But I think that your approach is wrong. In this moment, when you change the profile, you change as well the AWS cli path. You could have just one single script, not one per profile, and that script would look like:

@aws --no-verify-ssl %*

Best,

L.

On Tue, Oct 17, 2023 at 9:35 AM ArthurYang @.***> wrote:

this is the UI before click GEnerate Credentials button. [image: image] https://user-images.githubusercontent.com/113080404/275747889-d722ee3b-5161-4a24-9b6b-d1c40c62ce05.png

— Reply to this email directly, view it on GitHub https://github.com/coderazzi/aws_codeartifact_maven/issues/14#issuecomment-1765832620, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAH54IPQAPRMA7CNLFXLJCLX7YYDPAVCNFSM6AAAAAA6BL3U5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRVHAZTENRSGA . You are receiving this because you commented.Message ID: @.***>

Happy-ArthurYang commented 10 months ago

yes @aws --no-verify-ssl %* works fine, thank you for your help.