View Issue Details

This bug affects 1 person(s).
 10
IDProjectCategoryView StatusLast Update
19619Bug reportsAuthenticationpublic2024-07-15 11:20
Reporterriqcles Assigned ToDenisChenu  
PrioritynoneSeverityminor 
Status closedResolutionfixed 
Product Version6.5.x 
Summary19619: admin: bad password management url, with the publicurl variable
Description

Hello,

I have two servers:

One for the Front Office (server A):
'http://192.168.56.112/MonBeauCitronVert'

One for the Back Office (administration) (server B):
'http://192.168.56.113/admin'

I therefore entered the variable 'publicurl'=> 'http://192.168.56.112/MonBeauCitronVert' in my config.php file to separate the questionnaire construction and the responses from the internet users.

This works correctly.

However, I had not noticed that when I create new managers, the first link that is sent to create their password is the one for my server A.
This is also the case for the forgotten password procedure.

I have no problem sending invitations to answer the questionnaire (the invitations have the correct url for server A).

An additional variable (adminurl) could be set up?

Steps To Reproduce

Steps to reproduce

  1. Have two different URLs between the front and back office.
  2. Add the publicurl variable in the config.php file which points to the Office front
  3. Create a user in the back office
  4. Go to the new user's email
  5. click on the link to generate the new password

Expected result

the link redirects to the back office server

Actual result

the link redirects to the front office server

Another variation: with the forgotten password: same problem.

TagsNo tags attached.
Bug heat10
Complete LimeSurvey version number (& build)LimeSurvey Community Edition Version 6.5.13
I will donate to the project if issue is resolvedNo
BrowserFirefox / Chrome
Database type & versionPostgresql 15
Server OS (if known)centos 7.9
Webserver software & version (if known)Apache/2.4. (CentOS) PHP/8.2
PHP VersionPHP Version 8.2

Users monitoring this issue

DenisChenu

Activities

DenisChenu

DenisChenu

2024-06-20 08:54

developer   ~80400

According to forum post : there are a solution for 3.X version.
https://forums.limesurvey.org/forum/french-forum/156014-mot-de-passe-lien-d-urls-pour-les-gestionnaires-ko-avec-plusieurs-serveurs

Then : issue , not feature request.

My opinion : maybe add an optional settings adminurl for sending email for admin ?

DenisChenu

DenisChenu

2024-07-02 16:53

developer   ~80516

Did you try to set homeurl

  1. admin page must use homeurl
  2. token page must use publicurl
riqcles

riqcles

2024-07-03 17:30

reporter   ~80536

Yes using the homeulr variable, it works well with my VMs.
I'm going to contact my colleagues because there must be work on the mail servers, and with the proxy, I have no control.
But the idea of ​​an adminurl wouldn't be bad

DenisChenu

DenisChenu

2024-07-03 17:35

developer   ~80537

@tibor.pacalat I can take thçs one.

Need to udate PHPMailer for a option about url to be used + option here : https://github.com/LimeSurvey/LimeSurvey/blob/5bc06f5284736da00bddb44b8d98b406e6ea2fdb/application/core/LSYii_Controller.php#L166 to allow public OR admin url

DenisChenu

DenisChenu

2024-07-03 17:39

developer   ~80538

PS : we do not need an adminurl … if you ask password on http://192.168.56.113/admin : you need a http://192.168.56.113/admin url.

Just activate or deactivate usage of public url

adminurl can be interesting for Basic and Detailed Admin Notifications Email only.
But default on Basic and Detailed Admin Notifications Email must be current server too.

publicurl must be used only for token email: invitation, reminder and confirmation

DenisChenu

DenisChenu

2024-07-10 15:48

developer   ~80601

https://github.com/LimeSurvey/LimeSurvey/pull/3910

DenisChenu

DenisChenu

2024-07-11 17:37

developer   ~80607

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=36451

tibor.pacalat

tibor.pacalat

2024-07-11 17:38

administrator   ~80608

Tested and merged.

LimeBot

LimeBot

2024-07-15 11:20

administrator   ~80620

Fixed in Release 6.5.17+240715

Related Changesets

LimeSurvey: master 579ea596

2024-07-11 19:37

DenisChenu

Committer: GitHub


Details Diff
Fixed issue 19619: bad admin url, with the publicurl variable (03910) Affected Issues
19619
mod - application/models/services/PasswordManagement.php Diff File

Issue History

Date Modified Username Field Change
2024-06-19 23:55 riqcles New Issue
2024-06-20 08:52 DenisChenu Issue Monitored: DenisChenu
2024-06-20 08:52 DenisChenu Bug heat 0 => 2
2024-06-20 08:54 DenisChenu Note Added: 80400
2024-06-20 08:54 DenisChenu Bug heat 2 => 4
2024-07-02 16:53 DenisChenu Note Added: 80516
2024-07-03 17:30 riqcles Note Added: 80536
2024-07-03 17:30 riqcles Bug heat 4 => 6
2024-07-03 17:35 DenisChenu Note Added: 80537
2024-07-03 17:39 DenisChenu Note Added: 80538
2024-07-08 09:58 tibor.pacalat Assigned To => DenisChenu
2024-07-08 09:58 tibor.pacalat Status new => assigned
2024-07-10 15:41 gabrieljenik Assigned To DenisChenu => tibor.pacalat
2024-07-10 15:41 gabrieljenik Status assigned => ready for testing
2024-07-10 15:48 DenisChenu Note Added: 80601
2024-07-11 17:37 DenisChenu Changeset attached => LimeSurvey master 579ea596
2024-07-11 17:37 DenisChenu Note Added: 80607
2024-07-11 17:37 DenisChenu Assigned To tibor.pacalat => DenisChenu
2024-07-11 17:37 DenisChenu Resolution open => fixed
2024-07-11 17:38 tibor.pacalat Status ready for testing => resolved
2024-07-11 17:38 tibor.pacalat Note Added: 80608
2024-07-11 17:38 tibor.pacalat Bug heat 6 => 8
2024-07-15 11:20 LimeBot Note Added: 80620
2024-07-15 11:20 LimeBot Status resolved => closed
2024-07-15 11:20 LimeBot Bug heat 8 => 10