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

feat: Generic plugable "filters" #1660

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

sixcare
Copy link
Contributor

@sixcare sixcare commented Dec 15, 2021

Fixes #1475

Co-authored-by: Are Schjetne sixcare.as@gmail.com

Co-authored-by: Poul H. Sørensen poul@basefarm.no

Co-authored-by: Øystein Middelthun oystein@middelthun.no

Co-authored-by: Are Schjetne <sixcare.as@gmail.com>

Co-authored-by: Poul H. Sørensen <poul@basefarm.no>

Co-authored-by: Øystein Middelthun <oystein@middelthun.no>
@sixcare
Copy link
Contributor Author

sixcare commented Dec 15, 2021

#1475

@satterly satterly changed the title feat: Filters feat: Generic plugable "filters" Dec 16, 2021
@sixcare
Copy link
Contributor Author

sixcare commented Feb 7, 2023

missing filters https://github.com/sixcare/alerta/blob/feature/filters/alerta/database/backends/postgres/base.py#L146

@satterly satterly force-pushed the master branch 2 times, most recently from d73641a to 4cdfcfb Compare February 27, 2023 23:14
@sixcare sixcare marked this pull request as draft March 6, 2023 20:22
@sixcare sixcare marked this pull request as ready for review March 8, 2023 08:42
if filters:
return jsonify(
status='ok',
page=paging.page,

Check warning

Code scanning / CodeQL

Reflected server-side cross-site scripting Medium

Cross-site scripting vulnerability due to a
user-provided value
.
return jsonify(
status='ok',
page=paging.page,
pageSize=paging.page_size,

Check warning

Code scanning / CodeQL

Reflected server-side cross-site scripting Medium

Cross-site scripting vulnerability due to a
user-provided value
.
else:
return jsonify(
status='ok',
page=paging.page,

Check warning

Code scanning / CodeQL

Reflected server-side cross-site scripting Medium

Cross-site scripting vulnerability due to a
user-provided value
.
return jsonify(
status='ok',
page=paging.page,
pageSize=paging.page_size,

Check warning

Code scanning / CodeQL

Reflected server-side cross-site scripting Medium

Cross-site scripting vulnerability due to a
user-provided value
.
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.

Implementing delays and generic filter functionality
3 participants