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

doi-service does not process files with "lblx" file extension #416

Open
rsjoyner opened this issue Sep 26, 2023 · 1 comment
Open

doi-service does not process files with "lblx" file extension #416

rsjoyner opened this issue Sep 26, 2023 · 1 comment

Comments

@rsjoyner
Copy link

rsjoyner commented Sep 26, 2023

Checked for duplicates

No - I haven't checked

🐛 Describe the bug

Used this command to process 3 files each with extension of ".lblx".:

pds-doi-cmd release -N geo -s rsjoyner@jpl.nasa.gov -i /home/pds4/input/GEO/GEO_Mars_Sample_Dosier_20230926/ --no-review --force > /home/pds4/input/result_activate_no_review_GEO_Mars_Sample_Dosier_20230926.json

Received error: Please ensure the input is of the following type(s): .xml, .json

PDS4 approved use of ".lblx" for PDS4 XML labels. So, tools need (or will need) accept the new file extension for XML labels.

🕵️ Expected behavior

I expected the doi-service to process XML labels having "lblx" extension

📜 To Reproduce

  1. pds-doi-cmd release -N geo -s rsjoyner@jpl.nasa.gov -i /home/pds4/input/GEO/GEO_Mars_Sample_Dosier_20230926/ --no-review --force > /home/pds4/input/result_activate_no_review_GEO_Mars_Sample_Dosier_20230926.json
  2. Review processing stats

...

🖥 Environment Info

  • Version of this software [e.g. vX.Y.Z]
  • Operating System: [e.g. MacOSX with Docker Desktop vX.Y]
    ...

📚 Version of Software Used

No response

🩺 Test Data / Additional context

Interestingly, I attempted to attach one of the test ".lblx" files here and the file was rejected. The ".lblx" is not a supported file type....

🦄 Related requirements

🦄 #xyz

⚙️ Engineering Details

No response

@rsjoyner rsjoyner added bug Something isn't working needs:triage labels Sep 26, 2023
@tloubrieu-jpl tloubrieu-jpl removed bug Something isn't working needs:triage labels Nov 10, 2023
@tloubrieu-jpl
Copy link
Member

Hi @rsjoyner ,

We will look at this issue. You can add your .lblx file by zipping it before you attach it.

Thanks,

Thomas

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ToDo
Development

No branches or pull requests

3 participants