Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Redis sample fix to 3.2.2 - nitrogen #518

Closed
Microcom-Bjarne opened this issue Aug 26, 2021 · 1 comment
Closed

Redis sample fix to 3.2.2 - nitrogen #518

Microcom-Bjarne opened this issue Aug 26, 2021 · 1 comment
Assignees
@danieleteti
Labels
accepted Issue has been accepted and inserted in a future milestone bug
Milestone
3.2.2-nitrogen

Comments

@Microcom-Bjarne
Copy link
Contributor

delphimvcframework/samples/outputcachewithredis/WebModuleU.pas

Line 44 in a45cb13

MVC.Config['redis_connection_string'] := '127.0.0.1:6379';

MVC is readonly :-) so change the line to
Config['redis_connection_string'] := '127.0.0.1:6379';

and for the example to work another line is needed.
Config['redis_connection_key'] := '';
otherwise an exception is raised due to missing key from the cachecontroller.
@danieleteti danieleteti self-assigned this Aug 30, 2021
@danieleteti danieleteti added accepted Issue has been accepted and inserted in a future milestone bug labels Aug 30, 2021
@danieleteti danieleteti added this to the 3.2.2-nitrogen milestone Aug 30, 2021
danieleteti added a commit that referenced this issue Aug 30, 2021
@danieleteti
https://github.com/danieleteti/delphimvcframework/issues/518
04e687e
@danieleteti
Copy link
Owner

Thank you

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@danieleteti danieleteti

Labels
accepted Issue has been accepted and inserted in a future milestone bug
Projects
None yet
Milestone
3.2.2-nitrogen
Development

No branches or pull requests
2 participants
@danieleteti @Microcom-Bjarne