Skip to content
This repository has been archived by the owner on Dec 17, 2021. It is now read-only.

[CRASH] [BUG] [ERRORS] - Program "Semi-Crash" on Telephony #118

Closed
Nikolai558 opened this issue Aug 23, 2021 · 0 comments · Fixed by #122
Closed

[CRASH] [BUG] [ERRORS] - Program "Semi-Crash" on Telephony #118

Nikolai558 opened this issue Aug 23, 2021 · 0 comments · Fixed by #122
Assignees
Labels
bug Something isn't working HIGH PRIORITY Program Breaking Bugs or errors. Need to FIX asap.

Comments

@Nikolai558
Copy link
Owner

Describe the bug
Once NASR2SCT Hits the Telephony section, it goes straight to "Complete." No files are in the output directory.

I believe this is due to a website update. Telephony Website and inconsistencies in their HTML code and unaccounted error handling on NASR2SCT.

It is breaking on ' IRAQI PRESIDENTIAL '
image

It is breaking because we are looking for the closing html tag >. There is a better way to account for these, however it might require a refactor of this function entirely.
image

I am pretty sure I have a fix for this and will update as soon as possible.

@Nikolai558 Nikolai558 added bug Something isn't working HIGH PRIORITY Program Breaking Bugs or errors. Need to FIX asap. labels Aug 23, 2021
@Nikolai558 Nikolai558 self-assigned this Aug 23, 2021
Nikolai558 added a commit that referenced this issue Aug 23, 2021
This code fixes this issue. However, a new issue has a come from this.
For instance the .idIPF command works but there is a giant space in the telephony name.
Nikolai558 added a commit that referenced this issue Aug 23, 2021
Added a couple of checks to fix the telephony format update and hopefully future proofed it a little bit more.
Nikolai558 added a commit that referenced this issue Aug 28, 2021
@Nikolai558 Nikolai558 linked a pull request Aug 28, 2021 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working HIGH PRIORITY Program Breaking Bugs or errors. Need to FIX asap.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant