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

You cannot *not* request an MDN #51

Closed
kukel opened this issue Aug 2, 2018 · 7 comments
Closed

You cannot *not* request an MDN #51

kukel opened this issue Aug 2, 2018 · 7 comments
Assignees
Labels
Milestone

Comments

@kukel
Copy link

kukel commented Aug 2, 2018

Version: AS2-LIB 4.1.1-SNAPSHOT from 'master'

In AS2Client in the buildPartnership the AS2MDNTo is always set, so an MDN is always requested. You cannot unset it or populate the partnership in a different way

aPartnership.setAS2MDNTo (aSettings.getSenderAS2ID ());

So it is effectively impossible to not request an MDN (Yes, I know it is strange but there is one case configured like that live with a customer (yes for a test system, but still)

If I comment out the line above in the code

Maybe an additional method where the partnership is explicitly passed on, not via the settings would be a good addition.

@kukel kukel changed the title You cannot NOT request an MDN You cannot *NOT* request an MDN Aug 2, 2018
@kukel kukel changed the title You cannot *NOT* request an MDN You cannot *not* request an MDN Aug 2, 2018
@phax phax self-assigned this Aug 2, 2018
@kukel
Copy link
Author

kukel commented Aug 3, 2018

I don't think it is an enhancement but the result of the fix in #42 (9070fc2) which is effectively not fully correct and hence a 'bug' ;-)

@phax phax added bug and removed enhancement labels Aug 3, 2018
@phax phax added this to the 4.2.0 milestone Aug 6, 2018
@kukel
Copy link
Author

kukel commented Aug 6, 2018

There is a

@Nonnull
  public AS2ClientSettings setSenderData (@Nonnull final String sAS2ID,
                                          @Nonnull final String sEmailAddress,
                                          @Nonnull final String sKeyAlias)
  {

on the AS2ClientSettings, should the email address from that not be used instead of a fixed string in 71d92f3#diff-71c23868cad98a869a3e7fcd1c7dcef8R174

phax added a commit that referenced this issue Aug 6, 2018
@phax
Copy link
Owner

phax commented Aug 6, 2018

You know my code better than me :)

@phax
Copy link
Owner

phax commented Aug 6, 2018

Did you check whether it works as expected?

@kukel
Copy link
Author

kukel commented Aug 7, 2018

Will test later today.

@kukel
Copy link
Author

kukel commented Aug 7, 2018

Seems to be working!

@phax
Copy link
Owner

phax commented Aug 7, 2018

Great - thanks for testing.

@phax phax closed this as completed Aug 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants