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.

Sign up for GitHub

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

Jump to bottom

fix for #429: no session after PNR Retrieve failed #430

Merged
merged 1 commit into from
Feb 18, 2021
Merged

fix for #429: no session after PNR Retrieve failed #430

merged 1 commit into from
Feb 18, 2021

Conversation

DerMika
Copy link
Collaborator

@DerMika DerMika commented Feb 18, 2021

This should fix #429 . When catching the SoapFault, the post-message session check did not occur.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0002%) to 99.851% when pulling aa02eb8 on fix/429-no-session-after-pnr-retrieve-failed into e746900 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0002%) to 99.851% when pulling aa02eb8 on fix/429-no-session-after-pnr-retrieve-failed into e746900 on master.

@DerMika DerMika mentioned this pull request Feb 18, 2021
Closed
@DerMika DerMika merged commit 225ae42 into master Feb 18, 2021
@DerMika DerMika deleted the fix/429-no-session-after-pnr-retrieve-failed branch February 18, 2021 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

GETTING EMPTY SESSION ARRAY WHEN PNR_RETRIEVE CALL FAILED
2 participants
@DerMika @coveralls