Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[cpackget] NXP pack fails to download #412

Closed
brondani opened this issue Dec 18, 2024 · 1 comment
Closed

[cpackget] NXP pack fails to download #412

brondani opened this issue Dec 18, 2024 · 1 comment
Assignees
@brondani
Labels
bug Something isn't working

Comments

@brondani
Copy link
Collaborator

brondani commented Dec 18, 2024 •
edited
Loading

Downloading the pack NXP::MCXN947_DFP@19.0.0 fails:

$ cpackget add NXP::MCXN947_DFP@19.0.0 --verbose
...
D: Downloading https://mcuxpresso.nxp.com/cmsis_pack/repo/NXP.MCXN947_DFP.19.0.0.pack
D: bad status: 404 Not Found
E: "https://mcuxpresso.nxp.com/cmsis_pack/repo/NXP.MCXN947_DFP.19.0.0.pack": bad request

The curl command with and without user agent header seems to work:

curl -LO 'https://mcuxpresso.nxp.com/cmsis_pack/repo/NXP.MCXN947_DFP.19.0.0.pack' -H 'User-Agent: Go-http-client/1.1'
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 9844k  100 9844k    0     0  14.8M      0 --:--:-- --:--:-- --:--:-- 14.8M

However it fails for User-Agent: cpackget/v2.1.5
Possible regression: #359

Environment (please complete the following information):

  • OS: [e.g. windows]
  • Version [e.g. v2.1.5]
@brondani brondani added the bug Something isn't working label Dec 18, 2024
@brondani brondani self-assigned this Dec 18, 2024
@brondani brondani mentioned this issue Dec 18, 2024
Merged
5 tasks
jkrech pushed a commit that referenced this issue Dec 18, 2024
@brondani
[cpackget] Retry download without User-Agent header (#413)
2edf433 
## Fixes
<!-- List the issue(s) this PR resolves -->
- #412

## Changes
<!-- List the changes this PR introduces -->
- Resend the HTTP GET request without user agent header when the first
attempt with `User-Agent: cpackget/v<version>` fails with status code
`StatusNotFound`

## Checklist
<!-- Put an `x` in the boxes. All tasks must be completed and boxes
checked before merging. -->

- [x] 🤖 This change is covered by unit tests as required.
- [x] 🤹 All required manual testing has been performed.
- [x] 🛡️ Security impacts have been considered.
- [x] 📖 All documentation updates are complete.
- [x] 🧠 This change does not change third-party dependencies
@brondani brondani moved this to Done in CMSIS-Toolbox 2.8.0 Jan 14, 2025
@brondani brondani closed this as completed by moving to Done in CMSIS-Toolbox 2.8.0 Jan 14, 2025
@brondani
Copy link
Collaborator Author

Also fixed on NXP side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@brondani brondani

Labels
bug Something isn't working
Projects
CMSIS-Toolbox 2.8.0
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@brondani