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

Bugfix: ASA show interface - missing case handled for no values for duplex & speed #815

Conversation

starlightdreamer
Copy link
Contributor

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT

Cisco ASA show interface

SUMMARY

The line expected to have the format ...Duplex([parsed duplex value]), Auto-Speed([parsed speed value])
sometimes does not have any parenthesis after Duplex and Speed. When encountering this output, the current template fails. This change accounts for the possibility and assigns "" as the values.

Example of the expected duplex/speed line format:

	Auto-Duplex(Full-duplex), Auto-Speed(1000 Mbps)

Example of the unaccounted format:

        Full-Duplex, Auto-Speed

@starlightdreamer starlightdreamer force-pushed the ciscoAsa-showInterface-fixSpeed branch from 37252cf to d3e8b8b Compare October 25, 2020 21:11
@starlightdreamer starlightdreamer marked this pull request as ready for review October 25, 2020 21:11
@jmcgill298 jmcgill298 merged commit c340765 into networktocode:master Oct 26, 2020
@jmcgill298
Copy link
Contributor

Thanks @starlightdreamer

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

Successfully merging this pull request may close these issues.

2 participants