Skip to content

Commit

Permalink
Fix for RBAC (#52)
Browse files Browse the repository at this point in the history
* fix: typo and add license till 2024

* fix: Remove unnecessary info command which doesn't work with readonly users
  • Loading branch information
junatgit authored Feb 6, 2024
1 parent 3c8ad98 commit e447e58
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 11 deletions.
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,15 +39,15 @@ Args:
[<commands>] Redis commands and values
```

* `URI` URI to connect To. It follow the format of [the provisional IANA spec for Redis URLs](https://www.iana.org/assignments/uri-schemes/prov/redis), but with the option to denote a TLS secured connection with the protocol rediss:.
* `URI` URI to connect to. It follows the format of [the provisional IANA spec for Redis URLs](https://www.iana.org/assignments/uri-schemes/prov/redis), but with the option to denote a TLS secured connection with the protocol rediss:.

e.g. `INFO KEYSPACE`

Be aware of interactions with wild cards and special characters in the shell; quote and escape as appropriate.

## License

Redli is (c) IBM Corporation 2018. All rights reserved.
Redli is (c) IBM Corporation 2018-2024. All rights reserved.

Redli is released under the Apache 2 License.

Expand Down
9 changes: 0 additions & 9 deletions redli.go
Original file line number Diff line number Diff line change
Expand Up @@ -230,15 +230,6 @@ func main() {

sort.Strings(commandstrings)

reply, err := redis.String(conn.Do("INFO"))
if err != nil {
log.Fatal(err)
}

info := redisParseInfo(reply)

fmt.Printf("Connected to %s\n", info["redis_version"])

liner := liner.NewLiner()
defer liner.Close()

Expand Down

0 comments on commit e447e58

Please sign in to comment.