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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Upgrade to Redis 7 #4027

Closed
lenny-goodell opened this issue May 17, 2022 · 2 comments 路 Fixed by #4118
Closed

Upgrade to Redis 7 #4027

lenny-goodell opened this issue May 17, 2022 · 2 comments 路 Fixed by #4118
Assignees
@bnevis-i
Labels
enhancement New feature or request
Projects
Security WG
Milestone
Levski

Comments

@lenny-goodell
Copy link
Member

lenny-goodell commented May 17, 2022 •
edited
Loading

馃殌 Feature Request

Relevant Package [REQUIRED]

Core Data

Description [REQUIRED]

Need to stay up on latest version on Redis

Describe the solution you'd like

Use Redis 7.x, but we get the following error (secure mode) when subscribing to MessageBus that doesn't occur with Redis 6.2.x

level=ERROR ts=2022-05-17T23:48:01.4893991Z app=core-data source=subscriber.go:61 msg="NOPERM this user has no permissions to access one of the channels used as arguments"

Same error in Device Virtual when publishing and App Service when subscribing.

No issues in non-secure mode.

Describe alternatives you've considered

Stay on Redis 6.2.x
@lenny-goodell lenny-goodell added the enhancement New feature or request label May 17, 2022
@lenny-goodell lenny-goodell added this to New Issues in Core WG via automation May 17, 2022
@lenny-goodell lenny-goodell added this to the Levski milestone May 17, 2022
@lenny-goodell lenny-goodell added this to New Issues in Security WG via automation May 18, 2022
@lenny-goodell lenny-goodell removed this from New Issues in Core WG May 18, 2022
@lenny-goodell
Copy link
Member Author

@bnevis-i , FYI

@bnevis-i bnevis-i self-assigned this Jul 26, 2022
@bnevis-i bnevis-i moved this from New Issues to In progress in Security WG Jul 26, 2022
drkfmorton pushed a commit to drkfmorton/edgex-go that referenced this issue Aug 4, 2022
fix: Addressed permission error in redis v7 edgexfoundry#4027
8628ecf 
Signed-off-by: Kyle Morton <kylex.f.morton@intel.com>
@bnevis-i bnevis-i mentioned this issue Aug 4, 2022
Merged
5 tasks
@drkfmorton
Copy link
Contributor

The permission set created for the default user is missed access to all Pub/Sub channels

@bnevis-i bnevis-i moved this from In progress to QA/Code Review in Security WG Aug 4, 2022
drkfmorton pushed a commit to drkfmorton/edgex-go that referenced this issue Aug 4, 2022
fix: Updated permission edgexfoundry#4027
af2cb54 
Signed-off-by: Kyle Morton <kylex.f.morton@intel.com>
Security WG automation moved this from QA/Code Review to Done Aug 4, 2022
bnevis-i pushed a commit that referenced this issue Aug 4, 2022
@drkfmorton
fix: Addressed permission error in redis v7 #4027 (#4118)
153a892 
* fix: Addressed permission error in redis v7 #4027

Signed-off-by: Kyle Morton <kylex.f.morton@intel.com>

* fix: Updated permission #4027

Signed-off-by: Kyle Morton <kylex.f.morton@intel.com>

Co-authored-by: Kyle Morton <kylex.f.morton@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bnevis-i bnevis-i

Labels
enhancement New feature or request
Projects
Security WG
  
Done
Milestone
Levski
Development

Successfully merging a pull request may close this issue.

fix: Addressed permission error in redis v7 #4027 drkfmorton/edgex-go
3 participants
@drkfmorton @bnevis-i @lenny-goodell