Details
-
Type: Improvement
-
Status: Closed
-
Priority: Blocker
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: 5.0.0-b1
-
Component/s: None
-
Labels:None
-
Epic Link:
Description
The session and cookie names stored in the user's browser set by the application are set in the MinphpBridge service provider. Ideally, these should be configurable, so we should add them to the config file similar to the TTLs. We should also rename them to make them more unique to the application, that way they are less likely to conflict with any other applications on the web server that happen to also use the same session/cookie names.
- Create config values in /config/blesta.php for the session name and cookie name
- Update the MinphpBridge service provider to use these config values
- Change the values of the session/cookie names to be more unique to the application
- cookie "csid" should be "blesta_csid"
- session "sid" should be "blesta_sid"
- In order for an admin to upgrade Blesta to this version, they have to be logged-in, but they wouldn't be in any more once they uploaded the new files for the version since the MinphpBridge looks for and uses a session value that has not yet been created (since the upgrade script hasn't updated the config file). Because of this, we will most-likely need to fallback to the original values ("csid" and "sid") if no value is present in the config file.
Activity
Tyson Phillips (Inactive)
created issue -
Tyson Phillips (Inactive)
made changes -
Field | Original Value | New Value |
---|---|---|
Rank | Ranked higher |
Tyson Phillips (Inactive)
made changes -
Story Points | 2 |
Tyson Phillips (Inactive)
made changes -
Fix Version/s | 5.0.0-b1 [ 11107 ] |
Tyson Phillips (Inactive)
made changes -
Priority | Major [ 3 ] | Blocker [ 1 ] |
Tyson Phillips (Inactive)
made changes -
Epic Link | CORE-3003 [ 14161 ] |
Tyson Phillips (Inactive)
made changes -
Rank | Ranked lower |
Jonathan Reissmueller
made changes -
Sprint | 5.0.0 Sprint 2 [ 118 ] |
Jonathan Reissmueller
made changes -
Rank | Ranked lower |
Jonathan Reissmueller
made changes -
Assignee | Tyson Phillips [ tyson ] |
Jonathan Reissmueller
made changes -
Sprint | 5.0.0 Sprint 1 [ 118 ] | 5.0.0 Sprint 2 [ 119 ] |
Jonathan Reissmueller
made changes -
Assignee | Jonathan Reissmueller [ jonathan ] |
Automated transition triggered when Jonathan Reissmueller created a branch in Stash -
Status | Open [ 1 ] | In Progress [ 3 ] |
Automated transition triggered when Jonathan Reissmueller created pull request #995 in Stash -
Status | In Progress [ 3 ] | In Review [ 5 ] |
Resolution | Fixed [ 1 ] |
Jonathan Reissmueller
made changes -
Remaining Estimate | 0 minutes [ 0 ] | |
Time Spent | 1 hour, 3 minutes [ 3780 ] | |
Worklog Id | 14302 [ 14302 ] |
Abdy Franco
made changes -
Time Spent | 1 hour, 3 minutes [ 3780 ] | 1 hour, 23 minutes [ 4980 ] |
Worklog Id | 14305 [ 14305 ] |
Automated transition triggered when Jonathan Reissmueller merged pull request #995 in Stash -
Status | In Review [ 5 ] | Closed [ 6 ] |