HomePhorge

Plug the establishConnection() Lisk isolation hole

Description

Plug the establishConnection() Lisk isolation hole

Summary:
Currently you can still punch through Lisk isolation by calling
establishConnection(), and we do that all over the place. Rename getConnection()
to establishConnection() so that all existing callers are safe, and rename
establishConnection() to establishLiveConnection() so that it's not surprising
when this fails to stub in unit tests.

Not wedded to the name if anyone thinks "establishExternalConnection" or
something is clearer.

Test Plan:
Loaded site, browsed around, ran unit tests.

Reviewed By: aran
Reviewers: aran, tuomaspelkonen, jungejason
CC: aran
Differential Revision: 201

Details

Provenance
epriestleyAuthored on Apr 30 2011, 11:06 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP72e33c9e5a53: Fix a threading issue with Amazon SES
Branches
Unknown
Tags
Unknown

Event Timeline