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

GetListRow: fix tab separated values issue #28348

Merged
merged 2 commits into from Jul 20, 2023

Conversation

ilappe
Copy link
Contributor

@ilappe ilappe commented Jul 20, 2023

Related Issues

related:
https://jira-hq.paloaltonetworks.local/browse/XSUP-25540
#28066

Description

Server side issue that replace the tab character to 4 space cause the GetListRow script to not work as expected when the delimator of the list are tab

fix : replace \\t' with '\t' as now the tab separated values saved in server as \t` separated values

the list
image

call from Playbook
image

call from CLI
image

@ilappe ilappe requested a review from DeanArbel July 20, 2023 08:39
@ilappe ilappe self-assigned this Jul 20, 2023
Packs/CommonScripts/ReleaseNotes/1_12_5.md Outdated Show resolved Hide resolved
Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>
@ilappe ilappe merged commit 9c595e5 into master Jul 20, 2023
15 of 16 checks passed
@ilappe ilappe deleted the ilappe/fix_server_issue_with_tab_character branch July 20, 2023 11:19
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Aug 2, 2023
* fix tab issue in list

* Update Packs/CommonScripts/ReleaseNotes/1_12_5.md

Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>

---------

Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>
xsoar-bot pushed a commit to xsoar-contrib/content that referenced this pull request Aug 2, 2023
* fix tab issue in list

* Update Packs/CommonScripts/ReleaseNotes/1_12_5.md

Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>

---------

Co-authored-by: Dean Arbel <darbel@paloaltonetworks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants