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

docs: added Auth with Casbin in Apache APISIX blog #475

Merged
merged 16 commits into from
Aug 20, 2021
Merged

docs: added Auth with Casbin in Apache APISIX blog #475

merged 16 commits into from
Aug 20, 2021

Conversation

Serendipity96
Copy link

Changes:

docs: added Auth with Casbin in Apache APISIX blog

@netlify
Copy link

netlify bot commented Aug 18, 2021 •
edited
Loading

✔️ Deploy Preview for apisix ready!

🔨 Explore the source changes: 6ba4c75

🔍 Inspect the deploy log: https://app.netlify.com/sites/apisix/deploys/611f0bc2ff511b00089960db

😎 Browse the preview: https://deploy-preview-475--apisix.netlify.app

@juzhiyuan juzhiyuan requested review from bzp2010 and liuxiran August 19, 2021 02:47
bzp2010
bzp2010 reviewed Aug 19, 2021
View reviewed changes
Copy link
Contributor

@bzp2010 bzp2010 left a comment

Choose a reason for hiding this comment

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

Can file website/static/img/blog_img/2021-08-18-1.png use JPEG format (.jpg) to reduce size?

@hsluoyz hsluoyz mentioned this pull request Aug 19, 2021
Closed
moonming
moonming reviewed Aug 19, 2021
View reviewed changes
Copy link
Member

@moonming moonming left a comment

Choose a reason for hiding this comment

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

please add English version too

Serendipity96 and others added 2 commits August 19, 2021 17:02
@moonming
Update website/blog/Auth-with-Casbin-in-Apache-APISIX.md
17e4f1a 
Co-authored-by: Wen Ming <moonbingbing@gmail.com>
@moonming
Update website/blog/Auth-with-Casbin-in-Apache-APISIX.md
3efad8e 
Co-authored-by: Wen Ming <moonbingbing@gmail.com>
@Serendipity96
Copy link
Author

please add English version too

@yzeng25 Let's confirm the schedule of this article English version together.

@moonming
Copy link
Member

please add English version too

@yzeng25 Let's confirm the schedule of this article English version together.

Isn’t there a link to the English version at the end of the article?

@yzeng25
Copy link
Contributor

yzeng25 commented Aug 19, 2021

please add English version too

@yzeng25 Let's confirm the schedule of this article English version together.

Sure, I am looking at sometime next week ( no later than Wednesday) to get it done.

@Serendipity96
Copy link
Author

please add English version too

@yzeng25 Let's confirm the schedule of this article English version together.

Sure, I am looking at sometime next week ( no later than Wednesday) to get it done.

Maybe the link at the end of the article will help you.

@Serendipity96
Copy link
Author

please add English version too

@yzeng25 Let's confirm the schedule of this article English version together.

Isn’t there a link to the English version at the end of the article?

No, the content needs modify, it is the article‘s materials.

Serendipity96 and others added 3 commits August 20, 2021 09:55
@tzssangglass
Update website/blog/Auth-with-Casbin-in-Apache-APISIX.md
7a45e2a 
Co-authored-by: tzssangglass <tzssangglass@gmail.com>
@tzssangglass
Update website/blog/Auth-with-Casbin-in-Apache-APISIX.md
ea9e305 
Co-authored-by: tzssangglass <tzssangglass@gmail.com>
@tzssangglass
Update website/blog/Auth-with-Casbin-in-Apache-APISIX.md
6ba4c75 
Co-authored-by: tzssangglass <tzssangglass@gmail.com>
@liuxiran liuxiran merged commit 858e1c3 into apache:master Aug 20, 2021
@yzeng25 yzeng25 mentioned this pull request Aug 25, 2021
Merged
@yzeng25
Copy link
Contributor

yzeng25 commented Aug 26, 2021

English version of this blog is added in this pull request #510, it should be available by the end of this week.

1502shivam-singh pushed a commit to 1502shivam-singh/apisix-website that referenced this pull request Sep 19, 2021
@liuxiran @moonming
@tzssangglass @1502shivam-singh
docs: added Auth with Casbin in Apache APISIX blog (apache#475)
f9c07e2 
Co-authored-by: liuxiran <liuxiran@apache.org>
Co-authored-by: Wen Ming <moonbingbing@gmail.com>
Co-authored-by: tzssangglass <tzssangglass@gmail.com>
SkyeYoung pushed a commit to SkyeYoung/apisix-website that referenced this pull request Mar 3, 2022
@juzhiyuan
feat: added new plugin dependency (apache#475)
c81f0b9 
* feat: added new plugin dependency

* feat: omit shadow var

* feat: omit shadow var
SkyeYoung pushed a commit to SkyeYoung/apisix-website that referenced this pull request Mar 3, 2022
@LiteSun @Firstsawyou
@nic-chen @rapiz1 @liuxiran @juzhiyuan
fix: checkHostWithSSL with empty hosts (apache#502)
d61a5d9 
* feat: preview pluginchart (apache#456)

* feat: added pluginchart preview in route step4

* feat: change PluginChart to PluginOrchestration

* add tools to check ASF headers in code files. (apache#454)

* chore: remove unnecessary license file (apache#462)

* remove unnecessary license file

* remove unnecessary license file for `api7 dashboard`

* Increase the checking of .conf and .toml files. And delete the redundant configuration of the ASF-Release.cfg file. (apache#461)

* Increase the checking of .conf and .toml files.

* Skip the check of dag-to-lua-1.1/README.md file

* Put the "run Makefile" check before "get lua lib".

* Put the "run Makefile" check before "get lua lib".

* Put the "run Makefile" check before "get lua lib".

Co-authored-by: lin <lin@MBPro.local>

* ci: Add lint (apache#455)

Co-authored-by: Rapiz <code@rapiz.me>

* Feat: dashboard support route group (apache#433)

* feat: route group UI

* feat(route-ui): add routegroup when create route

* fix: add routegroup to route list

* fix: support add routegroup together with route

* fix: route path define error

* feat: trigger redeploy

* fix: update i18n key

Co-authored-by: 琚致远 <juzhiyuan@apache.org>

* feature:Independent license check operation file. (apache#465)

* feat: support ungroup in manager api (apache#460)

* feat: support ungroup in manager api

* fix: synchronize route group to route

* fix: update test case

* feat: add publish status to route in manager api (apache#450)

* feat: add publish status to route in manager api

* fix: update publish and offline route

* feat: add route publish test module

* fix: synchronize content for route publish/offline

* feat(route): add publish and offline to route (apache#451)

* feat(route): add publish and offline to route

* feat: trigger redeploy

* feat: trigger redeploy

* Update Create.tsx

* fix: update refer to the code review

* fix: init status default value

* fix: add a help msg to status Form.Item

Co-authored-by: 琚致远 <juzhiyuan@apache.org>

* feat: remove netlify (apache#472)

* fix: host should not be required (apache#477)

* feat: added new plugin dependency (apache#475)

* feat: added new plugin dependency

* feat: omit shadow var

* feat: omit shadow var

* fix: host should not be required (apache#479)

* fix: host should not be required

* feat: remove required rule

* feat: Route debug (apache#485)

* feat: new api get route and apisix url

* feat: online debug

* fix: update refer to the review, fix logical error as well

* fix: rename getRouteWithApisixUrl

* feat: update plugin (apache#482)

* feat: improve route (apache#483)

* feat: improve step1

* feat: improve step2

* feat: improve createStep4

* feat: improve transform

* fix: event loop

* feat: clean code

* fix: lost route_group info when enable redirect

* feat: UI improve

* fix: checkHostWithSSL with empty hosts

* Update Create.tsx

Co-authored-by: Firstsawyou <52862365+Firstsawyou@users.noreply.github.com>
Co-authored-by: nic-chen <33000667+nic-chen@users.noreply.github.com>
Co-authored-by: lin <lin@MBPro.local>
Co-authored-by: Rapiz <contact@rapiz.me>
Co-authored-by: Rapiz <code@rapiz.me>
Co-authored-by: liuxiran <belovedxixi@126.com>
Co-authored-by: 琚致远 <juzhiyuan@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bzp2010 bzp2010 bzp2010 left review comments

@tzssangglass tzssangglass tzssangglass left review comments

@liuxiran liuxiran liuxiran approved these changes

@moonming moonming moonming approved these changes

@KishaniKandasamy KishaniKandasamy KishaniKandasamy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@Serendipity96 @moonming @yzeng25 @liuxiran @bzp2010 @tzssangglass @KishaniKandasamy