Skip to content

scripts: add button to load script from clipboard - #7621

Merged
thc202 merged 1 commit into
zaproxy:mainfrom
thc202:scripts/add-script-clipboard
Aug 19, 2026
Merged

scripts: add button to load script from clipboard#7621
thc202 merged 1 commit into
zaproxy:mainfrom
thc202:scripts/add-script-clipboard

Conversation

@thc202

@thc202 thc202 commented Aug 5, 2026

Copy link
Copy Markdown
Member

Allow to create a script directly from the contents of the clipboard.

@psiinon

psiinon commented Aug 5, 2026

Copy link
Copy Markdown
Member

Logo
Checkmarx One – Scan Summary & Detailsbb007f68-6d61-4a33-bc74-ca7208ad9dd5

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to interact with Checkmarx PR Assistant.
Examples:
@Checkmarx how are you able to help me?
@Checkmarx rescan this PR

@ricekot

ricekot commented Aug 17, 2026

Copy link
Copy Markdown
Member

Do we really need this? 😅 Does not seem to save a lot of effort imo; users can create a new script and simply ⌘ + V into the script editor.

Maybe we can make the default script type in the dropdown in the new scripts dialog Stand Alone too, to be consistent with the load script dialog and to avoid the extra clicks.

@thc202

thc202 commented Aug 17, 2026

Copy link
Copy Markdown
Member Author

Maybe most users will not need it but I do miss this feature, I end up having to create a temp file just to try recordings (Zest does not allow to paste).

I'm good with changing the default type.

@ricekot

ricekot commented Aug 18, 2026

Copy link
Copy Markdown
Member

This is an unrelated bug, but the UI seems to hang when I try to load a Zest standalone script and select "Active Rules" as the "Type".

Maybe for another PR, but we should also consider adding some validation to the load script flow. Right now it is possible to load any text as a Zest script (both from file as well as clipboard), which then cannot be edited.

@kingthorin
kingthorin enabled auto-merge (rebase) August 18, 2026 20:53
@kingthorin
kingthorin disabled auto-merge August 18, 2026 20:53
Allow to create a script directly from the contents of the clipboard.

Signed-off-by: thc202 <thc202@gmail.com>
@thc202
thc202 force-pushed the scripts/add-script-clipboard branch from 519a914 to 76deddc Compare August 19, 2026 10:06
@thc202
thc202 enabled auto-merge August 19, 2026 10:07
@thc202
thc202 merged commit f7403d3 into zaproxy:main Aug 19, 2026
11 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 19, 2026
@thc202
thc202 deleted the scripts/add-script-clipboard branch August 19, 2026 10:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants