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

多站点策略缓存问题:使用数据库连接和表名作为缓存KEY #17

Closed
wants to merge 3 commits into from

Conversation

Tinywan
Copy link
Member

@Tinywan Tinywan commented Apr 30, 2021

多站点策略缓存问题:使用数据库连接和表名作为缓存KEY

@hsluoyz
Copy link
Member

hsluoyz commented Apr 30, 2021

@techoner

@leeqvip
Copy link
Member

leeqvip commented May 6, 2021

@Tinywan
It is recommended to add the cache configuration, refer to: Laravel-authz

@Tinywan
Copy link
Member Author

Tinywan commented May 6, 2021

@techoner

  • 1、目前该扩展在项目中是在多站点使用的,也就是说 connection(连接不同的数据库) 每次都是动态赋值的,而不是固定的写死的配置文件中的
  • 2、如果缓存是固定的配置文件,则导致每个站点的缓存都一样,从而最新的站点缓存不会被缓存。缓存是放在同一个Redis实例的同一个数据库中。
  • 3、多租户不同数据库权限配置,如:jiangxi.cabin_rules 缓存KEY,是缓存的江西casbin权限。beijing.cabin_rules 缓存KEY,是缓存的北京casbin权限。jiangxibeijing 标识不同的站点和不同的数据库,即不同的站点会有自己的数据库以及casbin策略规则表。

这是我整理笔记截图

Snipaste_2021-05-06_16-45-29

@leeqvip
Copy link
Member

leeqvip commented May 7, 2021

@Tinywan I think the use of caching configuration is more universal, and your caching strategy can be one of them.

@Tinywan
Copy link
Member Author

Tinywan commented May 7, 2021

@techoner
1、配置文件单站点是没什么问题。
2、现在是多站点多租户问题请问该如何解决这个缓存问题呢?哈哈

@leeqvip
Copy link
Member

leeqvip commented May 14, 2021

多站点可以有不同配置文件,配置不同的key即可,甚至可以配置不同的缓存驱动使用不同的redis库.

@Tinywan
Copy link
Member Author

Tinywan commented Jun 8, 2022

主要是缓存key目前是写死的 $rows = $this->model->cache('tauthz')

突然想到一个好办法:就是缓存key直接使用 $this->connection 即可,如果不为空,则$rows = $this->model->cache('tauthz'),否则$rows = $this->model->cache($this->connection)

@hsluoyz
Copy link
Member

hsluoyz commented Jun 8, 2022

@Tinywan 这个笔记可以开源出来么

@Tinywan
Copy link
Member Author

Tinywan commented Jun 8, 2022

这个一直不敢升级,没升级

@hsluoyz
Copy link
Member

hsluoyz commented Jun 8, 2022

@Tinywan 希望这个笔记可以开源出来,能贡献到官网文档就更好了

@Tinywan
Copy link
Member Author

Tinywan commented Jun 9, 2022

尽量哈

@Tinywan
Copy link
Member Author

Tinywan commented May 21, 2024

这个PR不给合并吗?

@hsluoyz
Copy link
Member

hsluoyz commented May 21, 2024

@Tinywan it has conflicts, can you fix them?

image

@Tinywan
Copy link
Member Author

Tinywan commented May 21, 2024

我看看

@Tinywan Tinywan closed this May 21, 2024
@hsluoyz
Copy link
Member

hsluoyz commented May 21, 2024

Replaced by: #46

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.

None yet

3 participants