Skip to content

Fix: Allow download of script output with "shell" type in .txt format#416

Closed
TarunaChaudhary-RA wants to merge 1 commit into
Telecominfraproject:masterfrom
routerarchitects:WIFI-14937-fix_script_command
Closed

Fix: Allow download of script output with "shell" type in .txt format#416
TarunaChaudhary-RA wants to merge 1 commit into
Telecominfraproject:masterfrom
routerarchitects:WIFI-14937-fix_script_command

Conversation

@TarunaChaudhary-RA
Copy link
Copy Markdown

@TarunaChaudhary-RA TarunaChaudhary-RA commented Jul 31, 2025

Fix: Allow download of script output with "shell" type in .txt format
JIRA: https://telecominfraproject.atlassian.net/browse/WIFI-14937
Summary of changes:
- This ensures the controller correctly processes files uploaded by APs when executing shell-type script commands, allowing successful download in .txt format.

JIRA: https://telecominfraproject.atlassian.net/browse/WIFI-14937

Summary of changes:
- This ensures the controller correctly processes files uploaded by APs when executing shell-type script commands, allowing successful download in `.txt` format.

Signed-off-by: Taruna Chaudhary <taruna.chaudhary@routerarchitects.com>
@TarunaChaudhary-RA TarunaChaudhary-RA force-pushed the WIFI-14937-fix_script_command branch from f619c5f to 331fddb Compare July 31, 2025 10:41
Copy link
Copy Markdown
Contributor

@carsten989 carsten989 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create a TIP branch and re-generate this PR based on that. We have problems with remote branches and our GITHUB actions.

is it possible to change the contenttype to be something less generic eg. text/scriptoutput so that we are not changing the behaviour for other existing responses?

@NavneetBarwal-RA
Copy link
Copy Markdown

Please create a TIP branch and re-generate this PR based on that. We have problems with remote branches and our GITHUB actions.

is it possible to change the contenttype to be something less generic eg. text/scriptoutput so that we are not changing the behaviour for other existing responses?

Hi Carsten ,

  1. I think we can ignore or decline this PR as the JIRA for this is still under discussion and changes are expected on APNOS side as well.
  2. I don't find any MIME type for text/script ,and it's the sender(APNOS) who sets the MIME type.
  3. Also making change of this sort won't affect behaviour for other existing responses as per my understanding.

@carsten989
Copy link
Copy Markdown
Contributor

Closing this PR as per comments above.

@carsten989 carsten989 closed this Aug 12, 2025
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

Successfully merging this pull request may close these issues.

3 participants