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

Add API endpoints for getting and setting UI options #320

Merged
merged 2 commits into from
Jan 3, 2019

Conversation

psiinon
Copy link
Member

@psiinon psiinon commented Dec 28, 2018

Fixes #319

@dscrobonia are these endpoints suitable for getting and setting the UI options?
The values are just strings, but they could be parsed as json / whatever if thats useful.
Theres no upper limit to the size of the values that can be set, but I'd suggest keeping them relatively small if possible.

@psiinon psiinon requested a review from dscrobonia December 28, 2018 17:34
@psiinon psiinon force-pushed the feature/api-ui-options branch from b1ae3e8 to 610556f Compare December 28, 2018 17:37
@psiinon psiinon force-pushed the feature/api-ui-options branch 2 times, most recently from f0a5dde to a6a3230 Compare January 2, 2019 16:20
@psiinon psiinon force-pushed the feature/api-ui-options branch from a6a3230 to eb3ffe1 Compare January 2, 2019 16:25
Copy link
Contributor

@dscrobonia dscrobonia left a comment

Choose a reason for hiding this comment

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

looks good for me at first pass. will play with it if you'd like to make sure, but looks good to me!

@psiinon
Copy link
Member Author

psiinon commented Jan 3, 2019

@dscrobonia can always change things once you have had a chance to use them :)

@psiinon psiinon merged commit 4e2dd40 into zaproxy:develop Jan 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants