Removes goroutines in search that were causing input issues on gemini cgi scripts #145

Closed
sloum wants to merge 0 commits from gemini-redirect-bug into develop
Owner

This solves an issue where a cgi script was handling a redirect. The search type of input was being handled as a goroutine and as such any input within it was wonky because there was already keylistening going on outside of it. I have removed the goroutines and the situation has been resolved.

Replicating this on a current build of Bombadillo can be done by referencing #144 . Then try it by building this branch. It definitely seems to solve the problem cleanly.

This solves an issue where a cgi script was handling a redirect. The search type of input was being handled as a goroutine and as such any input within it was wonky because there was already keylistening going on outside of it. I have removed the goroutines and the situation has been resolved. Replicating this on a current build of Bombadillo can be done by referencing #144 . Then try it by building this branch. It definitely seems to solve the problem cleanly.
sloum added the
bug
gemini
labels 2020-05-14 15:33:56 +00:00
Collaborator

Tested, this seems to work ok.

Tested, this seems to work ok.
Author
Owner

Closing this pull in favor of using a release branch ( #147 ).

Closing this pull in favor of using a release branch ( #147 ).
sloum closed this pull request 2020-05-16 05:34:25 +00:00

Pull request closed

Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sloum/bombadillo#145
No description provided.