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

Update parser.js #102

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update parser.js #102

wants to merge 1 commit into from

Conversation

zdrouse
Copy link

@zdrouse zdrouse commented Feb 7, 2024

sometimes nested objects in the config file are not indented and this causes the parser to break because the while loop doesn't exit and we reach the end of the object with no closing bracket detected.

the behavior can be observed in issue 99:
#99

I've only tested this with ucs argument and my conversion was able to complete successfully afterwards. I am not too certain if this change would impact anything else.

sometimes nested objects in the config file are not indented and this causes the parser to break because the while loop doesn't exit and we reach the end of the object with no closing bracket detected.

the behavior can be observed in issue 99:
f5devcentral#99
@zdrouse
Copy link
Author

zdrouse commented Feb 7, 2024

#99 might be a different issue than what I was facing actually

when HTTP_REQUEST {

if { [active_members [LB::server pool]] == 0 } {
        HTTP::respond 200 content $static::html
}
}

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.

None yet

1 participant