Skip to content

feat: support anonymous consumer#11917

Merged
Revolyssup merged 7 commits into
apache:masterfrom
Revolyssup:revolyssup/feat-anon-consumer
Jan 17, 2025
Merged

feat: support anonymous consumer#11917
Revolyssup merged 7 commits into
apache:masterfrom
Revolyssup:revolyssup/feat-anon-consumer

Conversation

@Revolyssup

Copy link
Copy Markdown
Contributor

Description

Currently, the built-in user authentication logic in APISIX is either successful authentication reflected to the corresponding Consumer object, or authentication failure directly returned to the caller authentication failure return.

Some users want a dummy consumer framed as "anonymous" consumer to be attached to the request when authentication fails so that certain rules (as plugins) defined in the "anonymous" consumer will be applied.

Fixes # (issue)

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@Revolyssup

Revolyssup commented Jan 15, 2025

Copy link
Copy Markdown
Contributor Author

multi auth plugin related tests to be fixed.

@dosubot dosubot Bot added size:XXL This PR changes 1000+ lines, ignoring generated files. enhancement New feature or request labels Jan 15, 2025
@Revolyssup
Revolyssup requested review from membphis, moonming, nic-6443 and shreemaan-abhishek and removed request for moonming January 16, 2025 08:03

@membphis membphis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Revolyssup
Revolyssup merged commit ce29823 into apache:master Jan 17, 2025
@Revolyssup
Revolyssup deleted the revolyssup/feat-anon-consumer branch January 17, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants