Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Teleporter can be redirected using tubes #67

Closed
oversword opened this issue Jan 16, 2021 · 4 comments · Fixed by joe7575/tubelib2#5 or #74
Closed

Teleporter can be redirected using tubes #67

oversword opened this issue Jan 16, 2021 · 4 comments · Fixed by joe7575/tubelib2#5 or #74

Comments

@oversword
Copy link
Contributor

Unsure why I couldn't find an existing issue for this one, it's been known on BLS for quite a while - to the point where we had to disable crafting teleporters, as this is very exploitable.

As usual I'm making this issue because I just fixed it! Expect a PR for this project & tubelib2 to come through immanently.

Basically, placing a tube next to a teleporter will redirect the teleporter as if it is a tube, this is annoying as it is, but after it has been redirected both the destination and origin teleporters can be removed without breaking the connection.

This effectively means you can teleport items anywhere with no cost.

To replicate:

  • Set up [ chest > pusher > teleporter ((airgap)) teleporter > chest ]
  • Either the pusher or the chest should not be high-perf, so you can use fewer items to test, sending them through one at a time will let us test for longer
  • Pair the teleporters together, put items in the first chest, switch the pusher on and confirm items are coming through
  • Place a tube next to the destination teleporter (at a face which does not have the hole in it)
  • You will now see items coming out of the tube, as if they have been redirected
  • Remove the destination teleporter, connection remains
  • Remove the source teleporter, connection remains

image

@oversword
Copy link
Contributor Author

@joe7575 you'll need to merge this one too to fix this
#68

It basically implements the new feature added to tubelib2 in the teleporters, so they refuse to connect on any side except the front

@oversword
Copy link
Contributor Author

👍

@joe7575
Copy link
Owner

joe7575 commented Jan 17, 2021

@oversword Thanks for your work to improve the mod!

@oversword
Copy link
Contributor Author

@joe7575 My pleasure! Honestly I just want my factory to work ;P
Seriously though, I've been wanting to do this for a while, I'm a pro at bug hunting and it's just silly waiting for someone else to fix it, that's what open source is for and that's why I'm playing MT not MC.
Thank YOU for being so attentive as a maintainer, you're the only one who seems to actually watch their projects!
You should be glad to know this is the last of the bugs as far as I know, though it would be naive to declare it flawless (:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants