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

Allow -bs flag for sendmail compatibility #22

Open
tobiasmuehl opened this issue Jun 2, 2017 · 1 comment
Open

Allow -bs flag for sendmail compatibility #22

tobiasmuehl opened this issue Jun 2, 2017 · 1 comment

Comments

@tobiasmuehl
Copy link

Just like -i, --long-i Ignored. This flag exists for sendmail compatibility. (default true),
add support for the -bs flag.

@tobiasmuehl tobiasmuehl changed the title Allow -bs flag for sendmail compatibility Allow -bs flag for sendmail compatibility Jun 2, 2017
@Potherca
Copy link

Potherca commented Aug 22, 2018

Looking at the manual for sendmail, there seem to be more flags that would also have to be ignore:

-Am
-Ac
-B
-bd
-bh
-bH
-bi
-bl
-bm
-bp
-bs
-bv
-C
-C
-F
-f -> Currently supported by mhsendmail
-G
-h
-I
-i -> This flag already exists for sendmail compatibility.
-L
-m
-N
-n
-oA
-O
-o7
-o8
-oi
-om
-o -> This flag already exists for sendmail compatibility.
-r
-R
-q
-qI
-qR
-qS
-t -> This flag already exists for sendmail compatibility.
-U
-V
-XV
-v -> Currently supported by mhsendmail
-X

Some of these flag also have a value, sometime without a space to separate the value.

I think the most sane approach would be to simply ignore any parameter other than those supported...

However, if that isn't desirable (as that would break with the current working of this library), I don't mind adding a pull-request that adds these parameters with the "Ignored. This flag exists for sendmail compatibility." message...

@ian-kent Can you please comment which approach would better suite you?
A. Adding every single parameter with ignore message
B. Changing parameter handling to use a white-list

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

No branches or pull requests

2 participants