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.

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

Already on GitHub? Sign in to your account

Issue with the message sent to owners to allow unsubscribing #469

Closed
bmarchal54 opened this issue Nov 8, 2018 · 8 comments · Fixed by #532
Closed

Issue with the message sent to owners to allow unsubscribing #469

bmarchal54 opened this issue Nov 8, 2018 · 8 comments · Fixed by #532
Labels
Milestone

Comments

@bmarchal54
Copy link
Contributor

In the message sent to list owners to allow unsubscription, in the url, it is the owner's address that is visible and not the address of the requester.
This is disturbing because the owner thinks, wrongly, that he subscribes.
If the 'mailto' link is sent, the correct address is visible.

Version

6.2.32

Installation method

Sympa was installed with rpm

Expected behavior

It should be the address of the subscriber that appears.

Actual behavior

Additional information

The template 'listowner_notification.tt2' produces the message on line 81.

@bmarchal54
Copy link
Contributor Author

bmarchal54 commented Nov 8, 2018

I propose the following modification of the file 'listowner_notification.tt2'.
It makes it possible to reach the list of subscriptions / unsubscribers waiting, and thus to make visible the addresses to be validated.

diff /usr/share/sympa/default/mail_tt2/listowner_notification.tt2 /etc/sympa/mail_tt2/listowner_notification.tt2
59c59
<   [% 'auth' | url_abs([keyauth,'add',list.name],{email=>user.email}) %]
---
>   [% 'subindex' | url_abs([list.name]) %]
77d76
< 
81c80
<   [% 'auth' | url_abs([keyauth,'del',list.name],{email=>user.email}) %]
---
>   [% 'sigindex' | url_abs([list.name]) %]

Cordially

@ikedas
Copy link
Member

ikedas commented Nov 9, 2018

Email in the link is address of approver, not subscriber nor requester. It's required by the feature.
And this link follows a confirmation page that has the link to sigindex/subindex page.

To clarify requester, you may add [% who %] in listowner_notification.tt2.

@bmarchal54
Copy link
Contributor Author

bmarchal54 commented Nov 9, 2018 via email

@ikedas
Copy link
Member

ikedas commented Nov 10, 2018

I see. web_tt2/confirm_action.tt2 also should display target address.

@ikedas
Copy link
Member

ikedas commented Jan 23, 2019

Hi @bmarchal54, if possible, could you please check if this patch will solve the problem?

  • This changes texts in templates, and translation would be updated later.

@ikedas ikedas added this to the 6.2.42 milestone Jan 23, 2019
@ikedas ikedas added the ready A PR is waiting to be merged. Close to be solved label Jan 23, 2019
@bmarchal54
Copy link
Contributor Author

bmarchal54 commented Jan 23, 2019 via email

@ikedas
Copy link
Member

ikedas commented Jan 24, 2019

Hello, I just tried your patch on our test server. When composing the sent URL, it is put an email address of one of the owners. But it is not necessarily this one who will do the action. Is this normal? cordially

I think it's normal. Who have done the action will be known by the other owner(s) if any.

ikedas added a commit that referenced this issue Jan 29, 2019
Issue with the message sent to owners to allow unsubscribing #469
@ikedas ikedas removed the ready A PR is waiting to be merged. Close to be solved label Jan 30, 2019
@ikedas
Copy link
Member

ikedas commented Feb 4, 2019

The PR has been included in 6.2.41b.1. Thanks for reporting bug and checking fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants