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.

Sign up for GitHub

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

Jump to bottom

Introduce dataobjects and or builders for Vert.x Web handlers #2519

Open
Tracked by #610
tsegismont opened this issue Nov 21, 2023 · 1 comment
Open
Tracked by #610

Introduce dataobjects and or builders for Vert.x Web handlers #2519

tsegismont opened this issue Nov 21, 2023 · 1 comment
Assignees
@tsegismont
Labels
enhancement
Milestone
5.0.0

Comments

@tsegismont
Copy link
Contributor

Review Vert.x Web handlers and determine which ones have several data params or callbacks.

These handlers should be reworked to provide a builder and/or a dataobject to simplify implementation.
@tsegismont tsegismont added this to the 5.0.0 milestone Nov 21, 2023
@tsegismont tsegismont self-assigned this Nov 21, 2023
@vietj vietj mentioned this issue Nov 24, 2023
Open
@tsegismont
Copy link
Contributor Author

tsegismont commented Dec 7, 2023 •
edited

Analyzed existing handlers and found these are good candidates

  • CSPHandler
    • data object
  • CSRFHandler
    • data object
  • ErrorHandler
    • data object
  • FaviconHandler
    • data object
  • FormLoginHandler
    • data object
  • RedirectAuthHandler
    • data object
  • TemplateHandler
    • data object
  • BodyHandler
    • data object
  • APIKeyHandler
    • data object
    • builder
  • AuthorizationHandler
    • builder
  • ChainAuthHandler
    • builder
  • CorsHandler
    • data object
  • JWTAuthHandler
    • data object
  • LoggerHandler
    • data object
    • builder
  • OAuth2AuthHandler
    • data object
    • builder
  • OtpAuthHandler
    • data object
    • builder
  • SessionHandler
    • data object
  • StaticHandler
    • data object
  • HealthCheckHandler
    • builder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tsegismont tsegismont

Labels
enhancement
Milestone
5.0.0
Development

No branches or pull requests
1 participant
@tsegismont