mistweaverco / bananas

Bananas🍌, Cross-Platform screen 🖥️ sharing 📡 made simple ⚡.
https://getbananas.net
MIT License
2.17k stars 43 forks source link

Bug: Join -Connect Button not clickable after pasting connection string #7

Closed alirezaalavi87 closed 4 days ago

alirezaalavi87 commented 4 days ago

v0.0.10

Hi. I start a session and then send the string that is obtained by clicking "copy my connection string" to my peer. when she puts the string in host connection string in join a session, It doesn't allow to press the connect button. The same happens when she creates session and I try to connect.

The connection string looks like this:

bananas://host?username=Banana%20Joe&token=eyJ0eXBlIjoib2ZmZXIiLCJzZHAiOiJ2PTBcclxubz0tIDQ4NzE5MzcyMjQzODYyNDQ2NjkgMiBJTiBJUDQgMTI3LjAuMC4xXHJcbnM9LVxyXG50PTAgMFxyXG5hPWdyb3VwOkJVTkRMRSAwIDEgMlxyXG5hPWV4dG1hcC1hbGxvdy1taXhlZFxyXG5hPW1zaWQtc2VtYW50aWM6IFdNUyBlY2VjMTc5NC1hYzE0LTRiODMtODA1ZS03ZGJkYTcxYTkzMDJcclxubT12aWRlbyA5IFVEUC9UTFMvUlRQL1NBVlBGIDk2IDk3IDEwMiAxMDMgMTA0IDEwNSAxMDYgMTA3IDEwOCAxMDkgMTI3IDEyNSAzOSA0MCA0NSA0NiA5OCA5OSAxMDAgMTAxIDExMiAxMTMgMTE0XHJcbmM9SU4gSVA0IDAuMC4wLjBcclxuYT1ydGNwOjkgSU4gSVA0IDAuMC4wLjBcclxuYT1pY2UtdWZyYWc6VVRrdFxyXG5hPWljZS1wd2Q6UXZ6a0VkVFhXMEJaU1N3WkZNWWs4U1lTXHJcbmE9aWNlLW9wdGlvbnM6dHJpY2tsZVxyXG5hPWZpbmdlcnByaW50OnNoYS0yNTYgREQ6NUU6MzU6NkU6Qzg6Qjg6Qzg6RUY6NkQ6QUM6NTQ6Qzk6NEQ6ODg6Mzc6OEQ6MjQ6MkQ6NDQ6MkI6M0M6MTQ6NTk6M0U6NTY6NjM6NjI6RDQ6RDA6NEU6MTI6MjZcclxuYT1zZXR1cDphY3RwYXNzXHJcbmE9bWlkOjBcclxuYT1leHRtYXA6MSB1cm46aWV0ZjpwYXJhbXM6cnRwLWhkcmV4dDp0b2Zmc2V0XHJcbmE9ZXh0bWFwOjIgaHR0cDovL3d3dy53ZWJydGMub3JnL2V4cGVykW1lbnRzL3J0cC1oZHJleHQvYWJzLXNlbmQtdGltZVxyXG5hPWV4dG1hcDozIHVybjozZ3BwOnZpZGVvLW9yaWVudGF0aW9uXHJcbmE9ZXh0bWFwOjQgaHR0cDovL3d3dy5pZXRmLm9yZy9pZC9kcmFmdl1ob2xtZXItcm1jYXQtdHJhbnNwb3J0LXdpZGUtY2MtZXh0ZW5zaW9ucy0wMVxyXG5hPWV4dG1hcDo1IGh0dHA6Ly93d3cud2VicnRjLm9yZy9leHBlcmltZW50cy9ydHAtaGRyZXh0L3BsYXlvdXQtZGVsYXlcclxuYT1leHRtYXA6NiBodHRwOi8vd3d3LndlYnJ0Yy5vcmcvZXhwZXJpbWVudHMvcnRwLWhkcmV4dC92aWRlby1jb250ZW50LXR5cGVcclxuYT1leHRtYXA6NyBodHRwOi8vd3d3LndlYnJ0Yy5vcmcvZXhwZXJpbWVudHMvcnRwLWhkcmV4dC92aWRlby10aW1pbmdcclxuYT1leHRtYXA6OCBodHRwOi8vd3d3LndlYnJ0Yy5vcmcvZXhwZXJpbWVudHMvcnRwLWhkcmV4dC9jb2xvci1zcGFjZVxyXG5hPWV4dG1hcDo5IHVybjppZXRmOnBhcmFtczpydHAtaGRyZXh0OnNkZXM6bWlkXHJcbmE9ZXh0bWFwOjEwIHVybjppZXRmOnBhcmFtczpydHAtaGRyZXh0OnNkZXM6cnRwLXN0cmVhbS1pZFxyXG5hPWV4dG1hcDoxMSB1cm46aWV0ZjpwYXJhbXM6cnRwLWhkcmV4dDpzZGVzOnJlcGFpcmVkLXJ0cC1zdHJlYW0taWRcclxuYT1zZW5kcmVjdlxyXG5hPW1zaWQ6ZWNlYzE3OTQtYWMxNC00YjgzLTgwNWUtN2RiZGE3MWE5MzAyIDBiODYzNDViLTk1NzUtNGExYi1iNTQ5LTdkZTZjMmZkZjlbNFxyXG5hPXJ0Y3AtbXV4XHJcbmE9cnRjcC1yc2l6ZVxyXG5hPXJ0cG1hcDo5NiBWUDgvOTAwMDBcclxuYT1ydGNwLWZiOjk2IGdvb2ctcmVtYlxyXG5hPXJ0Y3AtZmI6OTYgdHJhbnNwb3J0LWNjXHJcbmE9cnRjcC1mYjo5NiBjY20gZmlyXHJcbmE9cnRjcC1mYjo5NiBuYWNrXHJcbmE9cnRjcC1mYjo5NiBuYWNrIHBsaVxyXG5hPXJ0cG1hcDo5NyBydHgvOTAwMDBcclxuYT1mbXRwOjk3IGFwdD05NlxyXG5hPXJ0cG1hcDoxMDIgSDI2NC85MDAwMFxyXG5hPXJ0Y3AtZmI6MTAyIGdvb2ctcmVtYlxyXG5hPXJ0Y3AtZmI6MTAyIHRyYW5zcG9ydC1jY1xyXG5hPXJ0Y3AtZmI6MTAyIGNjbSBmaXJcclxuYT1ydGNwLWZiOjEwMiBuYWNrXHJcbmE9cnRjcC1mYjoxMDIgbmFjayBwbGlcclxuYT1mbXRwOjEwMiBsZXZlbC1hc3ltbWV0cnktYWxsb3dlZD0xO3BhY2tldGl6YXRpb24tbW9kZT0xO3Byb2ZpbGUtbGV2ZWwtaWQ9NDIwMDFmXHJcbmE9cnRwbWFwOjEwMyBydHgvOTAwMDBcclxuYT1mbXRwOjEwMyBhcHQ9MTAyXHJcbmE9cnRwbWFwOjEwNCBIMjY0LzkwMDAwXHJcbmE9cnRjcC1mYjoxMDQgZ29vZy1yZW1iXHJcbmE9cnRjcC1mYjoxMDQgdHJhbnNwb3J0LWNjXHJcbmE9cnRjcC1mYjoxMDQgY2NtIGZpclxyXG5hPXJ0Y3AtZmI6MTA0IG5hY2tcclxuYT1ydGNwLWZiOjEwNCBuYWNrIHBsaVxyXG5hPWZtdHA6MTA0IGxldmVsLWFzeW1tZXRyeS1hbGxvd2VkPTE7cGFja2V0aXphdGlvbi1tb2RlPTA7cHJvZmlsZS1sZXZlbC1pZD00MjAwMWZcclxuYT1ydHBtYXA6MTA1IHJ0eC85MDAwMFxyXG5hPWZtdHA6MTA1IGFwdD0xMDRcclxuYT1ydHBtYXA6MTA2IEgyNjQvOTAwMDBcclxuYT1ydGNwLWZiOjEwNiBnb29nLXJlbWJcclxuYT1ydGNwLWZiOjEwNiB0cmFuc3BvcnQtY2NcclxuYT1ydGNwLWZiOjEwNiBjY20gZmlyXHJcbmE9cnRjcC1mYjoxMDYgbmFja1xyXG5hPXJ0Y3AtZmI6MTA2IG5hY2sgcGxpXHJcbmE9Zm10cDoxMDYgbGV2ZWwtYXN5bW1ldHJ5LWFsbG93ZWQ9MTtwYWNrZXRpemF0aW9uLW1vZGU9MTtwcm9maWxlLWxldmVsLWlkPTQyZTAxZlxyXG5hPXJ0cG1hcDoxMDcgcnR4LzkwMDAwXHJcbmE9Zm10cDoxMDcgYXB0PTEwNlxyXG5hPXJ0cG1hcDoxMDggSDI2NC85MDAwMFxyXG5hPXJ0Y3AtZmI6MTA4IGdvb2ctcmVtYlxyXG5hPXJ0Y3AtZmI6MTA4IHRyYW5zcG9ydC1jY1xyXG5hPXJ0Y3AtZmI6MTA4IGNjbSBmaXJcclxuYT1ydGNwLWZiOjEwOCBuYWNrXHJcbmE9cnRjcC1mYjoxMDggbmFjayBwbGlcclxuYT1mbXRwOjEwOCBsZXZlbC1hc3ltbWV0cnktYWxsb3dlZD0xO3BhY2tldGl6YXRpb24tbW9kZT0wO3Byb2ZpbGUtbGV2ZWwtaWQ9NDJlMDFmXHJcbmE9cnRwbWFwOjEwOSBydHgvOTAwMDBcclxuYT1mbXRwOjEwOSBhcHQ9MTA4XHJcbmE9cnRwbWFwOjEyNyBIMjY0LzkwMDAwXHJcbmE9cnRjcC1mYjoxMjcgZ29vZy1yZW1iXHJcbmE9cnRjcC1mYjoxMjcgdHJhbnNwb3J0LWNjXHJcbmE9cnRjcC1mYjoxMjcgY2NtIGZpclxyXG5hPXJ0Y3AtZmI6MTI3IG5hY2tcclxuYT1ydGNwLWZiOjEyNyBuYWNrIHBsaVxyXG5hPWZtdHA6MTI3IGxldmVsLWFzeW1tZXRyeS1hbGxvd2VkPTE7cGFja2V0aXphdGlvbi1tb2RlPTE7cHJvZmlsZS1sZXZlbC1pZD00ZDAwMWZcclxuYT1ydHBtYXA6MTI1IHJ0eC85MDAwMFxyXG5hPWZtdHA6MTI1IGFwdD0xMjdcclxuYT1ydHBtYXA6MzkgSDI2NC85MDAwMFxyXG5hPXJ0Y3AtZmI6MzkgZ29vZy1yZW1iXHJcbmE9cnRjcC1mYjozOSB0cmFuc3BvcnQtY2NcclxuYT1ydGNwLWZiOjM5IGNjbSBmaXJcclxuYT1ydGNwLWZiOjM5IG5hY2tcclxuYT1ydGNwLWZiOjM5IG5hY2sgcGxpXHJcbmE9Zm10cDozOSBsZXZlbC1hc3ltbWV0cnktYWxsb3dlZD0xO3BhY2tldGl6YXRpb24tbW9kZT0wO3Byb2ZpbGUtbGV2ZWwtaWQ9NGQwMDFmXHJcbmE9cnRwbWFwOjQwIHJ0eC85MDAwMFxyXG5hPWZtdHA6NDAgYXB0PTM5XHJcbmE9cnRwbWFwOjQ1IEFWMS85MDAwMFxyXG5hPXJ0Y3AtZmI6NDUgZ29vZy1yZW1iXHJcbmE9cnRjcC1mYjo0NSB0cmFuc3BvcnQtY2NcclxuYT1ydGNwLWZiOjQ1IGNjbSBmaXJcclxuYT1ydGNwLWZiOjQ1IG5hY2tcclxuYT1ydGNwLWZiOjQ1IG5hY2sgcGxpXHJcbmE9Zm10cDo0NSBsZXZlbC1pZHg9NTtwcm9maWxlPTA7dGllcj0wXHJcbmE9cnRwbWFwOjQ2IHJ0eC85MDAwMFxyXG5hPWZtdHA6NDYgYXB0PTQ1XHJcbmE9cnRwbWFwOjk4IFZQOS85MDAwMFxyXG5hPXJ0Y3AtZmI6OTggZ29vZy1yZW1iXHJcbmE9cnRjcC1mYjo5OCB0cmFuc3BvcnQtY2NcclxuYT1ydGNwLWZiOjk4IGNjbSBmaXJcclxuYT1ydGNwLWZiOjk4IG5hY2tcclxuYT1ydGNwLWZiOjk4IG5hY2sgcGxpXHJcbmE9Zm10cDo5OCBwcm9maWxlLWlkPTBcclxuYT1ydHBtYXA6OTkgcnR4LzkwMDAwXHJcbmE9Zm10cDo5OSBhcHQ9OThcclxuYT1ydHBtYXA6MTAwIFZQOS85MDAwMFxyXG5hPXJ0Y3AtZmI6MTAwIGdvb2ctcmVtYlxyXG5hPXJ0Y3AtZmI6MTAwIHRyYW5zcG9ydC1jY1xyXG5hPXJ0Y3AtZmI6MTAwIGNjbSBmaXJcclxuYT1ydGNwLWZiOjEwMCBuYWNrXHJcbmE9cnRjcC1mYjoxMDAgbmFjayBwbGlcclxuYT1mbXRwOjEwMCBwcm9maWxlLWlkPTJcclxuYT1ydHBtYXA6MTAxIHJ0eC85MDAwMFxyXG5hPWZtdHA6MTAxIGFwdD0xMDBcclxuYT1ydHBtYXA6MTEyIHJlZC85MDAwMFxyXG5hPXJ0cG1hcDoxMTMgcnR4LzkwMDAwXHJcbmE9Zm10cDoxMTMgYXB0PTExMlxyXG5hPXJ0cG1hcDoxMTQgdWxwZmVjLzkwMDAwXHJcbmE9c3NyYy1ncm91cDpGSUQgMTUzMTI3MDI5NCAzMTcwNDgzMDc0XHJcbmE9c3NyYzoxNTMxMjcwMjk0IGNuYW1lOk8yL3B6QzJGNTdLc1JoSUNcclxuYT1zc3JjOjE1MzEyNzAyOTQgbXNpZDplY2VjMTc5NC1hYzE0LTRiODMtODA1ZS03ZGJkYTcxYTkzMDIgMGI4NjM0NWItOTU3NS00YTFiLWI1NDktN2RlNmMyZmRmOWQ0XHJcbmE9c3NyYzozMTcwNDgzMDc0IGNuYW1lOk8yL3B6QzJGNTdLc1JoSUNcclxuYT1zc3JjOjMxNzA0ODMwNzQgbXNpZDplY2VjMTc5NC1hYzE0LTRiODMtODA1ZS03ZGJkYTcxYTkzMDIgMGI4NjM0NWItOTU3NS00YTFiLWI1NDktN2RlNmMyZmRmOWQ0XHJcbm09YXVkaW8gOSBVRFAvVExTL1JUUC9TQVZQRiAxMTEgNjMgOSAwIDggMTMgMTEwIDEyNlxyXG5jPUlOIElQNCAwLjAuMC4wXHJcbmE9cnRjcDo5IElOIElQNCAwLjAuMC4wXHJcbmE9aWNlLXVmcmFnOlVUa3RcclxuYT1pY2UtcHdkOlF2emtFZFRYVzBCWlNTd1pGTVlrOFNZU1xyXG5hPWljZS1vcHRpb25zOnRyaWNrbGVcclxuYT1maW5nZXJwcmludDpzaGEtMjU2IEREOjVFOjM1OjZFOkM4OkI4OkM4OkVGOjZEOkFDOjU0OkM5OjREOjg4OjM3OjhEOjI0OjJEOjQ0OjJCOjNDOjE0OjU5OjNFOjU2OjYzOjYyOkQ0OkQwOjRFOjEyOjI2XHJcbmE9c2V0dXA6YWN0cGFzc1xyXG5hPW1pZDoxXHJcbmE9ZXh0bWFwOjE0IHVybjppZXRmOnBhcmFtczpydHAtaGRyZXh0OnNzcmMtYXVkaW8tbGV2ZWxcclxuYT1leHRtYXA6MiBodHRwOi8vd3d3LndlYnJ0Yy5vcmcvZXhwZXJpbWVudHMvcnRwLWhkcmV4dC9hYnMtc2VuZC10aW1lXHJcbmE9ZXh0bWFwOjQgaHR0cDovL3d3dy5pZXRmLm9yZy9pZC9kcmFmdC1ob2xtZXItcm1jYXQtdHJhbnNwb3J0LXdpZGUtY2MtZXh0ZW5zaW9ucy0wMVxyXG5hPWV4dG1hcDo5IHVybjppZXRmOnBhcmFtczpydHAtaGRyZXh0OnNkZXM6bWlkXHJcbmE9c2VuZHJlY3ZcclxuYT1tc2lkOmVjZWMxNzk0LWFjMTQtNGI4My04MDVlLTdkYmRhNzFhOTMwMiBiNjQzOGY4OC0xMjBhLTQ5ZWItODExNS1hZjdlYmQ3Y2ZhNTRcclxuYT1ydGNwLW11eFxyXG5hPXJ0cG1hcDoxMTEgb3B1cy80ODAwMC8yXHJcbmE9cnRjcC1mYjoxMTEgdHJhbnNwb3J0LWNjXHJcbmE9Zm10cDoxMTEgbWlucHRpbWU9MTA7dXNlaW5iYW5kZmVjPTFcclxuYT1ydHBtYXA6NjMgcmVkLzQ4MDAwLzJcclxuYT1mbXRwOjYzIDExMS8xMTFcclxuYT1ydHBtYXA6OSBHNzIyLzgwMDBcclxuYT1ydHBtYXA6MCBQQ01VLzgwMDBcclxuYT1ydHBtYXA6OCBQQ01BLzgwMDBcclxuYT1ydHBtYXA6MTMgQ04vODAwMFxyXG5hPXJ0cG1hcDoxMTAgdGVsZXBob25lLWV2ZW50LzQ4MDAwXHJcbmE9cnRwbWFwOjEyNiB0ZWxlcGhvbmUtZXZlbnQvODAwMFxyXG5hPXNzcmM6MTI0NjQ3ODYyNCBjbmFtZTpPMi9wekMyRjU3S3NSaElDXHJcbmE9c3NyYzoxMjQ2NDc4NjI0IG1zaWQ6ZWNlYzE3OTQtYWMxNC00YjgzLTgwNWUtN2RiZGE3MWE5MzAyIGI2NDM4Zjg4LTEyMGEtNDllYi04MTE1LWFmN2ViZDdjZmE1NFxyXG5tPWFwcGxpY2F0aW9uIDkgVURQL0RUTFMvU0NUUCB3ZWJydGMtZGF0YWNoYW5uZWxcclxuYz1JTiBJUDQgMC4wLjAuMFxyXG5hPWljZS11ZnJhZzpVVGt0XHJcbmE9aWNlLXB3ZDpRdnprRWRUWFcwQlpTU3daRk1ZazhTWVNcclxuYT1pY2Utb3B0aW9uczp0cmlja2xlXHJcbmE9ZmluZ2VycHJpbnQ6c2hhLTI1NiBERDo1RTozNTo2RTpDODpCODpDODpFRjo2RDpBQzo1NDpDOTo0RDo4ODozNzo4RDoyNDoyRDo0NDoyQjozQzoxNDo1OTozRTo1Njo2Mzo2MjpENDpEMDo0RToxMjoyNlxyXG5hPXNldHVwOmFjdHBhc3NcclxuYT1taWQ6MlxyXG5hPXNjdHAtcG9ydDo1MDAwXHJcbmE9bWF4LW1lc3NhZ2Utc2l6ZToyNjIxNDRcclxuIn0%3D
gorillamoe commented 4 days ago

What's your OS?

alirezaalavi87 commented 4 days ago

@gorillamoe one is linux one is win11

gorillamoe commented 4 days ago

Can you try the following:

Paste it and just remove the b from bananas:// and add it again.

Maybe the event listener for paste is messed up.

gorillamoe commented 4 days ago

Works for me, can you check that the URL is as pasted here? when shared via office365 it's altered to something else and not the original anymore. The one you pasted works for me with no problems.