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.

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

Investigate invalid PDSC downloads #154

Open
flit opened this issue Nov 23, 2020 · 2 comments
Open

Investigate invalid PDSC downloads #154

flit opened this issue Nov 23, 2020 · 2 comments

Comments

@flit
Copy link
Member

flit commented Nov 23, 2020

(Reported by @rapgenic in #121.)

In the cmsis-pack-manager local PDSC repository, a few .pdsc files are definitely not PDSC XML. Instead they contain strange html/js code. Is this a result of a missed redirection/missing resource, or is it simply invalid configuration on the part of the silicon vendors?

List of questionable files:

  • ABOV.CM0_DFP.1.0.4.pdsc
  • ABOV.CM3_DFP.1.2.5.pdsc
  • MindMotion.MM32x031_DFP.1.0.0.pdsc
  • MindMotion.MM32x103_DFP.1.1.0.pdsc
  • Redpine.RS13100_DFP.1.0.0.pdsc
  • Redpine.RS14100_DFP.1.0.6.pdsc
  • SiliconLabs.ARTIKBG1_DFP.5.7.0.pdsc
  • SiliconLabs.ARTIKMG1_DFP.5.7.0.pdsc
  • Sinowealth.SH32Fxxx_DFP.1.0.0.pdsc
@rapgenic
Copy link

Also, when updating the index, a few warnings are reported (which I originally ignored, being them warnings), like (note that there are many others and a lot of them are related to functioning packs which probably contain little errors):

Nov 23 17:13:27.339 WARN tried to parse element "package" from element "html"
Nov 23 17:13:27.454 WARN the XML is invalid, an element was wrongly closed
Nov 23 17:13:27.464 WARN Only Comment, CDATA and DOCTYPE nodes can start with a '!'

Which suggest that the library is recognising those files as problematic.

@flit
Copy link
Member Author

flit commented Aug 30, 2021

Most of the listed PDSCs are actually returning 404 Not Found responses. Because of #162 and #155, the html content of the error message sent by the server gets written to the .pdsc file.

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

No branches or pull requests

2 participants