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

Fix(filterExecuteAttributes): Utilizing memory address pointers #294

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

People-Sea
Copy link
Contributor

Fix tests get define START_TIME'

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug labels Jul 2, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 2, 2024
@zds-s zds-s enabled auto-merge (squash) July 2, 2024 14:58
@zds-s zds-s disabled auto-merge July 2, 2024 14:58
@zds-s zds-s merged commit e094254 into mineadmin:master Jul 2, 2024
7 of 12 checks passed
zds-s added a commit that referenced this pull request Jul 9, 2024
* Fix(filterExecuteAttributes): Utilizing memory address pointers (#294)

* LoginListener.php: 修复因为多点登录复用同一个token导致该账号最近一次登录的设备登出就无法获得正确的用户在线情况 (#295)

SystemUserService.php: 1. 重写 token-key 正则匹配 2. kickUser 保证获取到所有token,一次性全部下线。 3. hasTokenBlack 只判断传入token所在scene的情况,否则会导致判断多重scene而导致在线用户监控列表出错。

* FIX : 用户信息password字段隐藏.避免安全隐患. (#297)

* LoginListener.php: 修复因为多点登录复用同一个token导致该账号最近一次登录的设备登出就无法获得正确的用户在线情况
SystemUserService.php: 1. 重写 token-key 正则匹配 2. kickUser 保证获取到所有token,一次性全部下线。 3. hasTokenBlack 只判断传入token所在scene的情况,否则会导致判断多重scene而导致在线用户监控列表出错。

* fix 用户返回信息会出现password字段,虽然是加密过的字段,但依旧存在安全隐患。故隐藏。

* UserAuthService.php 修复因为隐藏password导致user模型 toarray之后无法获取password导致登录流程出错的问题

* UserAuthService.php fix:获取主键value 而不是主键 key (Tab 按快了)

* Fix: 优化 is_in_container 函数(#298)

---------

Co-authored-by: PeopleSea <70972819+People-Sea@users.noreply.github.com>
Co-authored-by: ShaBaoFa <wlfpanda1012@gmail.com>
Co-authored-by: clq321 <405292878@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug lgtm This PR has been approved by a maintainer Setting size:XS This PR changes 0-9 lines, ignoring generated files. System
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants