Handle case where the frontend url already contains query parameters

[PythonCoderAS]

To my knowledge, if there is a "?" in the URL, everything after is a query parameter, so we check for a "?" and if one does exist we use "&" instead of "?".

Fix #365

[CLAassistant]

All committers have signed the CLA.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[mostafa]

How about using urllib? https://stackoverflow.com/a/64117554/6999563

[PythonCoderAS]

How about using urllib? https://stackoverflow.com/a/64117554/6999563

Not necessarily because the frontend url might be a relative URL (ie "/login" and not "http://my-domain.com/login") which won't parse under urlparse

Coverage dropped. It would be nice to add a test.

[mostafa]

@PythonCoderAS It would be nice if you could add a test as well.

[github-actions]

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

[PythonCoderAS]

Bump

[github-actions]

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

[PythonCoderAS]

Hi, this is still an active PR with a test case, could this please be reviewed?