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

Async mdn question #57

Open
ihudedi opened this issue Aug 27, 2018 · 5 comments
Open

Async mdn question #57

ihudedi opened this issue Aug 27, 2018 · 5 comments

Comments

@ihudedi
Copy link

ihudedi commented Aug 27, 2018

Hi Philip,
I saw in code whem using async mdn we store the mic in mdmpendinginfo folder and in the file we set the mic and mdmpending file.
My question what are you doing with this file beacuse the file isnt created and you are trying to delete him when mic is matched.
Another issuewhen you delete mdmpendinginfo file you print to log the directory from mdmpending and not from mdmpendinginfo.
Thanks,
Itay

@phax phax self-assigned this Aug 27, 2018
@phax phax added the question label Aug 27, 2018
@phax phax added this to the 4.2.0 milestone Aug 27, 2018
@phax phax added the bug label Aug 27, 2018
@ihudedi
Copy link
Author

ihudedi commented Aug 27, 2018

In addition maybe ability to set the mdmpendinginfo fron the as2clientsettings.currently i modifed the as2message to extend and set those params.

@phax phax removed this from the 4.2.0 milestone Aug 27, 2018
@stale
Copy link

stale bot commented May 14, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label May 14, 2019
@phax phax added pinned and removed wontfix labels May 15, 2019
@phax
Copy link
Owner

phax commented Nov 6, 2019

That one can be closed, as "pendingmdninfo" and "pendingmdn" can be set on the message processor - right?

@ihudedi
Copy link
Author

ihudedi commented Nov 10, 2019

Please the code when you store data to those folders and when you retrieve from there or delete.
Seems like redundant code.
Thanks,
Itay

@phax
Copy link
Owner

phax commented Jun 6, 2021

I think this one is resolved in the meantime. Feel free to open it up again, if you feel that something needs to change.

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

No branches or pull requests

2 participants