Allow safe_mode to be manually set in the Email Library #1531

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
4 participants
@dixy
Contributor

dixy commented Jun 25, 2012

For example, on Infomaniak ini_get('safe_mode') returns false so the mail() function uses its fifth parameter (additional_parameters).
However Infomaniak does not allow this parameter. I have to set safe_mode to on manually to get mail function working.

@narfbg

This comment has been minimized.

Show comment Hide comment
@narfbg

narfbg Jun 25, 2012

Contributor

What's Infomaniak and how does it not allow it? It doesn't allow the function argument at all or it doesn't specifically allow the -f parameter?

Contributor

narfbg commented Jun 25, 2012

What's Infomaniak and how does it not allow it? It doesn't allow the function argument at all or it doesn't specifically allow the -f parameter?

@dixy

This comment has been minimized.

Show comment Hide comment
@dixy

dixy Jun 26, 2012

Contributor

Infomaniak seems to not allow the last argument of mail() (they have some restrictions to prevent spam).

See: http://expressionengine.com/archived_forums/viewreply/681593/

Contributor

dixy commented Jun 26, 2012

Infomaniak seems to not allow the last argument of mail() (they have some restrictions to prevent spam).

See: http://expressionengine.com/archived_forums/viewreply/681593/

@jjaffeux

This comment has been minimized.

Show comment Hide comment
@jjaffeux

jjaffeux Jun 26, 2012

Contributor

narfbg, infomaniak is an hosting provider : http://www.infomaniak.com/

Contributor

jjaffeux commented Jun 26, 2012

narfbg, infomaniak is an hosting provider : http://www.infomaniak.com/

@narfbg

This comment has been minimized.

Show comment Hide comment
@narfbg

narfbg Jun 26, 2012

Contributor

I don't think that it's worth adding such stuff due to one hosting company's restrictive policies. But even if so, I don't think that changing the pseudo safe mode is the proper thing - other stuff may depend on it (now, or in the future). Something more tightly related to -f maybe?

Contributor

narfbg commented Jun 26, 2012

I don't think that it's worth adding such stuff due to one hosting company's restrictive policies. But even if so, I don't think that changing the pseudo safe mode is the proper thing - other stuff may depend on it (now, or in the future). Something more tightly related to -f maybe?

@ckdarby

This comment has been minimized.

Show comment Hide comment
@ckdarby

ckdarby Oct 3, 2012

Contributor

@narfbg I agree that this is an edge case; I would much rather see this pushed out to someone's MY_Email than to be pulled into CI.

Contributor

ckdarby commented Oct 3, 2012

@narfbg I agree that this is an edge case; I would much rather see this pushed out to someone's MY_Email than to be pulled into CI.

@narfbg

This comment has been minimized.

Show comment Hide comment
@narfbg

narfbg Oct 4, 2012

Contributor

It's been a while since we commented on this one ...

@dixy Have you figured another way around this?

Contributor

narfbg commented Oct 4, 2012

It's been a while since we commented on this one ...

@dixy Have you figured another way around this?

@dixy

This comment has been minimized.

Show comment Hide comment
@dixy

dixy Oct 7, 2012

Contributor

No, I just made a MY_Email class where I removed all references to safe_mode.

Contributor

dixy commented Oct 7, 2012

No, I just made a MY_Email class where I removed all references to safe_mode.

@narfbg

This comment has been minimized.

Show comment Hide comment
@narfbg

narfbg Oct 7, 2012

Contributor

Okay then, closing this one. I don't think that -f has anything to do with safe mode anyway.

Contributor

narfbg commented Oct 7, 2012

Okay then, closing this one. I don't think that -f has anything to do with safe mode anyway.

@narfbg narfbg closed this Oct 7, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment