Skip to content

flask_demo: add guard.policy/guard.protect decorator examples + expand docs - #207

Open
AaryanInzalkar wants to merge 1 commit into
0-Shimanshu:mainfrom
AaryanInzalkar:feat/add-flask-demo
Open

flask_demo: add guard.policy/guard.protect decorator examples + expand docs#207
AaryanInzalkar wants to merge 1 commit into
0-Shimanshu:mainfrom
AaryanInzalkar:feat/add-flask-demo

Conversation

@AaryanInzalkar

Copy link
Copy Markdown
Contributor

Summary

Adds decorator-based guard integration examples to flask_demo (@guard.policy(),
@guard.protect(), @guard.hooks.on_event) and expands documentation with curl
examples, an adv CLI section, and real verification commands.

Why

The merged flask_demo (#187) only used configure_routes() for guard integration
and never demonstrated the decorator-based API. In the fastapi_demo PR, the
maintainer explicitly requested @guard.policy() and @guard.protect() usage
before merging. This brings flask_demo up to the same standard so both
integration styles (declarative vs. decorator-based) are documented in one place.

Scope

What kind of change is this?

  • bug fix
  • documentation
  • tests
  • TUI / CLI improvement
  • framework integration
  • signal change
  • configuration or route-policy change

Verification

How did you verify the change?

Commands run:

python app.py
curl -X POST http://127.0.0.1:5000/auth/login/ -H "Content-Type: application/json" -d '{}'
curl -X POST http://127.0.0.1:5000/admin/action/ -H "Content-Type: application/json" -d '{}'

Notes:

  • I ran the relevant tests locally
  • I updated docs if behavior changed
  • I updated screenshots if the TUI changed materially

Signal-specific notes

Signal type:

  • hard signal (trackA)
  • soft signal (trackB)
  • not applicable

Source, reproducer, or proof: N/A
Benign case checked: N/A
Performance note: N/A

TUI notes

  • before/after screenshots attached
  • not applicable

Local-only file check

  • I did not accidentally stage local-only files such as adiuvare.yaml or local conftest.py changes

Related issue

Closes #

@AaryanInzalkar

Copy link
Copy Markdown
Contributor Author

@Wahid7852 @0-Shimanshu i have suggested a few changes as I was asked please check

@AaryanInzalkar

Copy link
Copy Markdown
Contributor Author

@Wahid7852 @0-Shimanshu Heyy could you pls check

@Wahid7852

Copy link
Copy Markdown
Collaborator

wheres the issue about it :D
also why tf u still on about GSSoC. its such a scam

@AaryanInzalkar

AaryanInzalkar commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

@Wahid7852 #14 is the related issue regarding this but now it has disappeared into thin air 😭😭

I don't have anything else to work on man, searching for an internship rn
Hook me up if you have smth no 👉👈

@AaryanInzalkar

Copy link
Copy Markdown
Contributor Author

@Wahid7852 Brodda just laughed at me :< atleast reply

@AaryanInzalkar

Copy link
Copy Markdown
Contributor Author

@Wahid7852 Brodda

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.

2 participants