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

Släpp (DISCARD) meddelanden 'från' spexadresser som inte lagts in som… #55

Merged
merged 4 commits into from
Jun 26, 2023

Conversation

anargam
Copy link
Collaborator

@anargam anargam commented Jun 17, 2023

… extramailadresser. Skicka inte mailstudset till spexmaillistan.

@anargam anargam requested a review from Karlinde June 17, 2023 13:44
Nothing hidden here and others added 2 commits June 17, 2023 15:45
@anargam
Copy link
Collaborator Author

anargam commented Jun 17, 2023

Hjärnsläpp. Tog bort en (beroende på postfix- och annats hantering av konstiga adresser) potentiell sårbarhet. Nothing to see here, move along. Vi vill kanske ha något med strukturerad loggning framöver.

@anargam
Copy link
Collaborator Author

anargam commented Jun 17, 2023

Enklare tester bekräftar att detta

  • stoppar from:-tricket, som ger mailstudsar till en spexlista,
  • inte stoppar korrekta mail (föga förvånande, det ligger efter kollen av giltig mailadress),
  • inte stoppar maillistor som hänvisar till varandra (itindirektion -> it_mailtest exempelvis).
  • inte stoppar korrekta mailstuds (adresser som inte finns).

@Rovanion
Copy link
Member

Potentiellt dum fråga men jag minns inte riktigt: Har vi användare som har inkorgar med adresser som slutar med studentspex.se?

lissmilter.py Outdated Show resolved Hide resolved
lissmilter.py Outdated Show resolved Hide resolved
lissmilter.py Show resolved Hide resolved
@anargam
Copy link
Collaborator Author

anargam commented Jun 19, 2023

Potentiellt dum fråga men jag minns inte riktigt: Har vi användare som har inkorgar med adresser som slutar med studentspex.se?

Inte mig veterligen. Däremot möjligen avsändaradresser, och mailalias eller maillistor (för mottagande).

@anargam anargam dismissed Karlinde’s stale review June 21, 2023 14:24

Rensat i kod enligt önskemål. Den större förändringen fortsätter i länkat ärende. Kollat och testat representanter ur den nämnda felklassen.

@anargam anargam linked an issue Jun 21, 2023 that may be closed by this pull request
@Rovanion
Copy link
Member

Generellt så är jag för att slå in det här koden på master så löser vi senare problem när de dyker upp typ.

@anargam anargam requested review from Karlinde and Rovanion June 22, 2023 10:56
@Rovanion Rovanion merged commit af2a95c into master Jun 26, 2023
@Rovanion Rovanion deleted the bouncybouncy branch June 26, 2023 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inga mailstuds till spexlistor
3 participants