Skip to content

fix: java init stuck in main and param parser broken for values with =#444

Merged
pradeeban merged 1 commit into
ControlCore-Project:devfrom
avinxshKD:fix/java-init-and-param-parser
Feb 22, 2026
Merged

fix: java init stuck in main and param parser broken for values with =#444
pradeeban merged 1 commit into
ControlCore-Project:devfrom
avinxshKD:fix/java-init-and-param-parser

Conversation

@avinxshKD
Copy link
Copy Markdown
Contributor

Fixes #424

moved init (iport, oport, params, maxtime) from main() into a static {} block so the class actually initializes on load
replaced the broken replaceAll("=", "':'") fallback with a proper

  • parseParams() that splits on first = only mirrors
  • concore_base.parse_params
image

@avinxshKD
Copy link
Copy Markdown
Contributor Author

Hey @pradeeban the old fallback was doing replaceAll("=", "':'") which nukes every = in the value... so something like url=https://x.com?a=1 turns into {'url':'https://x.com?a':'1'} and literalEval rightfully throws. split on first = only fixes it, same way parse_params in python already handles it. also moved init out of main() since nobody calls main() when importing the class from another file.

Let me know if you want changes in this file or any other Java parts. Goodnight

@pradeeban pradeeban merged commit cc6ba38 into ControlCore-Project:dev Feb 22, 2026
6 checks passed
@pradeeban
Copy link
Copy Markdown
Member

@avinxshKD Java implementation was mostly just a toy. No wonder it was buggy.

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