mirror of
https://github.com/ProxymanApp/atlantis.git
synced 2026-04-26 08:26:04 +03:00
[GH-ISSUE #119] duplicated connection working in an office #71
Labels
No labels
Done
Done
Windows
bug
bug
bug
enhancement
enhancement
enhancement
good first issue
hacktoberfest
pull-request
wontfix
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/atlantis#71
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @chrisknapp98 on GitHub (Nov 15, 2022).
Original GitHub issue: https://github.com/ProxymanApp/atlantis/issues/119
Originally assigned to: @NghiaTranUIT on GitHub.
hi, me and my colleagues are experiencing duplicated connections from atlantis to Proxyman when we are working at the office. Thus, we receive the same network request multiple times with the exact same time, duration etc.
I couldn't find the root of the issue, but filtering in NetServiceTransport#connectToService(_ service: NetService) also gets the job done. Maybe it could be a change to be considered for a future release. Let's see if I manage to create a PR
A problem could be that with that change devices which have the same name couldn't connect both over the network to atlantis
@NghiaTranUIT commented on GitHub (Nov 15, 2022):
Do you specify the Computer Name when starting the Atlantis Service?
By doing it, Atlantis only connects to this computer, not your colleague's computers.
You can find a name in the Atlantis Windows.
@NghiaTranUIT commented on GitHub (Nov 15, 2022):
and what Atlantis library version you're using 🤔 ?
@chrisknapp98 commented on GitHub (Nov 15, 2022):
version 1.19.0
@chrisknapp98 commented on GitHub (Nov 15, 2022):
The issue is not that it connects to my colleagues computers. No matter if I specify the device in start call, atlantis still keeps connecting multiple times to my Proxyman client. Sometimes it doesn't but often the conection is there like 5 times
@chrisknapp98 commented on GitHub (Nov 15, 2022):
Okay I haven't got any permissions to push a new branch. But my fix is in line 226 after the guard let hostName to add
@chrisknapp98 commented on GitHub (Nov 15, 2022):
ohh I'm just noticing that splitting at a dot and taking index 0 only works with our devices because the host name of our devices always is the serial number with the .local suffix
@NghiaTranUIT commented on GitHub (Nov 15, 2022):
If you don't mind, please add a checkpoint to this line
github.com/ProxymanApp/atlantis@a761f2c8a2/Sources/Transporter.swift (L229)and see how many print output.
If it connect to the same Macbook multiple times, we can see multiple outputs.
@chrisknapp98 commented on GitHub (Nov 15, 2022):
exactly! That's how I knew atlantis connected to the same device multiple times. Without the guard block I added, this print with the exact same hostName can be seen multiple times.
I think this also especially happens when there are like 3+ colleagues in the office which are running Proxyman on their mac
@NghiaTranUIT commented on GitHub (Nov 15, 2022):
@chrisknapp98 can you try this branch https://github.com/ProxymanApp/atlantis/pull/120?
I refactor a little bit and prevent connecting multiple times to a single Host. I suppose that it can fix the bug.
How to install
Cocoapod
SPM
7ee8312b13a6b872a0068bdf11bfc66511614c9f@chrisknapp98 commented on GitHub (Nov 15, 2022):
seems to be working 👍
I censored the names because it would be the serial numbers
@NghiaTranUIT commented on GitHub (Nov 16, 2022):
Thanks. Please play around with this update. If it's still working fine, I will release the fix this week 👍
@NghiaTranUIT commented on GitHub (Nov 19, 2022):
Just a friendly reminder that I've bumped new version 1.20.0, which contains the fix.
However, if you've encountered this bug https://github.com/ProxymanApp/atlantis/issues/121 , Let's try the Proxyman beta build 👍