setup-node icon indicating copy to clipboard operation
setup-node copied to clipboard

feat: support private mirrors

Open marco-ippolito opened this issue 11 months ago • 5 comments

Fixes: https://github.com/actions/setup-node/issues/1150 Fixes: https://github.com/actions/setup-node/issues/730

marco-ippolito avatar Mar 06 '25 09:03 marco-ippolito

the failures dont seem related to this change are they?

marco-ippolito avatar Mar 07 '25 13:03 marco-ippolito

Hello @marco-ippolito, Thank you for your pull request! We are currently reviewing it. To resolve the check failures, could you please rebase your branch with the main branch and push the changes.

aparnajyothi-y avatar Mar 25 '25 04:03 aparnajyothi-y

@aparnajyothi-y thanks, rebased

marco-ippolito avatar Mar 25 '25 12:03 marco-ippolito

Hello marco-ippolito, The fallback log message could be clarified to reduce potential confusion between falling back directly from Node and falling back to downloading from the Node mirror URL. Currently, it states: "Falling back to download directly from Node" for both scenarios. A slight adjustment could provide clearer context in the logs. Please feel free to reach out if you have any concerns. Screenshot 2025-03-27 at 6 46 41 PM

aparnajyothi-y avatar Mar 27 '25 13:03 aparnajyothi-y

@aparnajyothi-y change the message to Not found in manifest. Falling back to download directly from 'your-mirror'

marco-ippolito avatar Mar 27 '25 13:03 marco-ippolito

I wonder if we can merge this

marco-ippolito avatar Mar 28 '25 12:03 marco-ippolito

Hello @marco-ippolito, With the latest commit, the log message now correctly displays the mirror URL when provided, which is the expected behavior. However, in the default fallback scenario (when no mirror URL is provided), the log message currently shows: "Falling back to download directly from." Instead, it should remain as it was previously, showing: "Falling back to download directly from Node." This log message should remain unchanged for the default fallback behavior. Please feel free to reach out if you need further clarification or have any concerns. Screenshot 2025-03-28 at 9 18 07 PM

aparnajyothi-y avatar Mar 28 '25 15:03 aparnajyothi-y

@aparnajyothi-y thanks I have updated the code

marco-ippolito avatar Mar 31 '25 14:03 marco-ippolito

Hello @marco-ippolito, With the latest commit, the log message still showing: "Falling back to download directly from" Instead, it should remain as it was previously, showing: "Falling back to download directly from Node" This log message should remain unchanged for the default fallback behavior. Please feel free to reach out if you need further clarification or have any concerns. image

aparnajyothi-y avatar Apr 01 '25 13:04 aparnajyothi-y

@aparnajyothi-y amended

marco-ippolito avatar Apr 01 '25 13:04 marco-ippolito