Commit Graph

537 Commits

Author SHA1 Message Date
github-actions[bot]
ee76a2f7e0 Automated: Update Podfile.lock (#4661)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4661

Reviewed By: ivanmisuno

Differential Revision: D44995374

Pulled By: passy

fbshipit-source-id: 4c93a3750da5ae9977e62451e0bf7f3717b8c233
2023-04-14 04:10:57 -07:00
Richard Howell
60f4bc1a65 enable -Wstrict-prototypes
Summary:
The 14.3 toolchain includes `-Wstrict-prototypes` in `-Wall`. Add it to the default warning list to ensure the code is buildable with the 14.3 toolchain.

Please forward fix any build breakage introduced by this change.

Reviewed By: NSProgrammer

Differential Revision: D44371491

fbshipit-source-id: 257c7f7748b079a34ffd5c041339d518bef9d1e5
2023-03-28 10:48:57 -07:00
github-actions[bot]
b45d56d9fe Automated: Update Podfile.lock (#4624)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4624

Reviewed By: passy

Differential Revision: D44301473

Pulled By: aigoncharov

fbshipit-source-id: 8edee27e69b441aed959e9555a1fe5e1d1582e88
2023-03-23 05:06:55 -07:00
github-actions[bot]
81d1070ccc Automated: Update Podfile.lock (#4604)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4604

Reviewed By: mweststrate

Differential Revision: D44131877

Pulled By: passy

fbshipit-source-id: b1ce5dd121c077e88db2c9d916edd79b7d383f18
2023-03-16 13:20:54 -07:00
github-actions[bot]
6eacbe87cc Automated: Update Podfile.lock (#4528)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4528

Reviewed By: ivanmisuno

Differential Revision: D43974467

Pulled By: passy

fbshipit-source-id: b2a0b3539add658f59221acadc6701bdcae3238c
2023-03-10 06:27:37 -08:00
Andrey Goncharov
8f2ae1cc02 Back out "Remove FBDefines from sonar"
Summary:
Original commit changeset: ff6db8e5e4bb

Original Phabricator Diff: D43408140 (b0a6b0f430)

It breaks the build for our iOS Sample app.- https://github.com/facebook/flipper/actions/runs/4212014740

Reviewed By: lblasa

Differential Revision: D43466846

fbshipit-source-id: e06a6f57b7fff08791d875e20825fbe5e0db0b37
2023-02-21 10:34:20 -08:00
Adam Ernst
b0a6b0f430 Remove FBDefines from sonar
Summary: This appears to be unreferenced and it constantly confuses our header autocomplete. Remove.

Reviewed By: d16r

Differential Revision: D43408140

fbshipit-source-id: ff6db8e5e4bbda394197168680cf8d15b07666c3
2023-02-18 08:16:02 -08:00
Lorenzo Blasa
c1badd1a29 RCMountable -> CKMountable (#4522)
Summary:
Pull Request resolved: https://github.com/facebook/flipper/pull/4522

^

A codemod changed the name of `CKMountable` for the OSS ComponentKit integration, this is a breaking change as we are pinned to a specific Cocoapods version of CK.

Reviewed By: passy

Differential Revision: D43303489

fbshipit-source-id: 4d8ad472735414ca8e67a8ce4409efe84aab3e94
2023-02-15 03:55:28 -08:00
Lorenzo Blasa
e9c0a459dd Remove connection timeout
Summary:
This change mainly removes the connection timeout period as it may introduce a race condition in which slow server connections get terminated potentially creating connection loops.

Also, suspend and wait for all operations to complete once a socket is disconnected.

Reviewed By: ivanmisuno

Differential Revision: D43048252

fbshipit-source-id: 64c28a3d3d2fd4e065084d5f55a17444385c07e0
2023-02-07 05:55:11 -08:00
Lorenzo Blasa
2ba167f899 Back out "Clear handlers on disconnect"
Summary:
^

Revert as to validate this is not causing regressions: T143523262

Reviewed By: passy

Differential Revision: D42800560

fbshipit-source-id: 8db61454eabfdb259637bb97c2bb4754984ecf6f
2023-01-27 04:07:37 -08:00
Lorenzo Blasa
f44feb69e0 SKDescriptorMapper should be a weak reference
Summary:
^

SKDescriptorMapper owns the SKNodeDescriptor instances. SKNodeDescriptor instances should only have a weak reference to the mapper as to avoid retain cycles.

Reviewed By: passy

Differential Revision: D42673698

fbshipit-source-id: 8c98709b28fc3c711dc56c179c7c362417fa1f9d
2023-01-23 03:47:41 -08:00
Lorenzo Blasa
764e94503e Clear handlers on disconnect
Summary:
A few changes that should make our connect/disconnect more robust:

* Certificate provider handler should be set directly to and managed by the policy.
* Instantiate the socket once, synchronously on the connect method. Explicit deallocation once, synchronously on the disconnect method.
* Clear handlers on disconnect after clearing the delegate.
* Wait for the operation queue to drain before returning.

Reviewed By: passy

Differential Revision: D42664724

fbshipit-source-id: bd482acbb64a9bc9e36fb3418d4c81afa2109305
2023-01-23 03:45:59 -08:00
Lorenzo Blasa
8dd5b57444 Diagnostic controller improvement
Summary:
The diagnostics controller is not the best crafted UIViewController. It has many things that should be improved.

- This diff just sets the background of the root view to white. At least, that will ensure that its text content is always shown if the screen background happens to be black (same as font colour).
- Correct offset calculation, used for scrolling. Effectively, only set it if the content to be displayed no longer fits in the scrollview viewport.

Reviewed By: LukeDefeo

Differential Revision: D41876904

fbshipit-source-id: e2a89d8f6001e5b626c8df1d0832e77783999b81
2022-12-13 09:49:46 -08:00
Lorenzo Blasa
91effaea44 Update SampleSwift as to remove CK dependencies
Summary: ^

Reviewed By: LukeDefeo

Differential Revision: D41868295

fbshipit-source-id: 52f594fe1817459e99dab25bf9c87eebdef2168e
2022-12-09 02:22:28 -08:00
Kfir Schindelhaim
b55a35dd0f Rename CKMountable -> RCMountable
Summary: ^^

Reviewed By: adamjernst

Differential Revision: D41496622

fbshipit-source-id: aa4ce00ef5f05f58935e30d1570fbaa8b57c55d7
2022-12-01 13:20:37 -08:00
Lorenzo Blasa
1406e291ee Send with raw parameters
Summary:
On Android we already had an API to send raw arguments i.e. raw json as params.

This just adds feature parity on iOS.

Reviewed By: LukeDefeo

Differential Revision: D41433777

fbshipit-source-id: abd47f987b9e2b451100e81acf6fea61cd876807
2022-11-28 09:45:53 -08:00
Lorenzo Blasa
c95c59342e Remove CK from Sample (#4321)
Summary:
Pull Request resolved: https://github.com/facebook/flipper/pull/4321

^

Reviewed By: antonk52

Differential Revision: D41434959

fbshipit-source-id: 35fe20a2e04af1d6011f0fb9c82aee4eb504fdc7
2022-11-21 07:09:43 -08:00
facebook-flipper-bot
ff282630be Automated: Update Podfile.lock (#4271)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4271

Reviewed By: passy

Differential Revision: D40755392

Pulled By: aigoncharov

fbshipit-source-id: f296fe36535ff8744ddee6b6f9d2575d71c429f9
2022-10-27 05:30:53 -07:00
facebook-flipper-bot
2bde26391d Automated: Update Podfile.lock (#4238)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4238

Reviewed By: aigoncharov

Differential Revision: D40636695

Pulled By: nikoant

fbshipit-source-id: fda37eba37dd286affe977d7a9046945c8aaaab8
2022-10-24 08:05:38 -07:00
facebook-flipper-bot
10d71a321c Automated: Update Podfile.lock (#4235)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4235

Reviewed By: antonk52

Differential Revision: D40471242

Pulled By: aigoncharov

fbshipit-source-id: c8028f0b4be05ff5b9156443c52a86def4d1e179
2022-10-18 08:34:04 -07:00
facebook-flipper-bot
8a2bbddb6a Automated: Update Podfile.lock (#4210)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4210

Reviewed By: mweststrate

Differential Revision: D40341306

Pulled By: aigoncharov

fbshipit-source-id: 1383835dce4ede179cb49598f0b57840712975a9
2022-10-13 06:57:31 -07:00
facebook-flipper-bot
9bfcd2ed46 Automated: Update Podfile.lock (#4176)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4176

Reviewed By: cekkaewnumchai

Differential Revision: D40095627

Pulled By: cekkaewnumchai

fbshipit-source-id: c37e6c148fb346a20e7c39a112ba7a441a81e386
2022-10-05 04:32:55 -07:00
facebook-flipper-bot
abb4c3c8cf Automated: Update Podfile.lock (#4156)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4156

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

Static Docs Site previews have moved into the custom phabricator field "Static Docs", and will no longer modify test plans after 5th October 2022.

Reviewed By: antonk52

Differential Revision: D39931809

Pulled By: antonk52

fbshipit-source-id: eb234e760e9a6be321b7290ff20b5232e9840fed
2022-10-03 03:19:24 -07:00
facebook-flipper-bot
cb75be7495 Automated: Update Podfile.lock (#4128)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4128

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D39687101/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D39687101/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: antonk52

Differential Revision: D39687101

Pulled By: aigoncharov

fbshipit-source-id: 568a6740033cf37bcc86492eefbcb247c7bf1948
2022-09-21 05:56:35 -07:00
facebook-flipper-bot
25f40605a8 Automated: Update Podfile.lock (#4025)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4025

Reviewed By: lblasa

Differential Revision: D38979176

Pulled By: cekkaewnumchai

fbshipit-source-id: d455bbdb5390340cd2b310d9be5045b088dbb994
2022-08-25 02:39:27 -07:00
facebook-flipper-bot
e316f51874 Automated: Update Podfile.lock (#4011)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/4011

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D38785235/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D38785235/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: passy

Differential Revision: D38785235

Pulled By: mweststrate

fbshipit-source-id: fc71d3d61940e769a2c5fe25a39984f6a327d909
2022-08-17 12:12:33 -07:00
Marcelo Lopez Ruiz
b7c38556ce Name the Flipper scheduler/threads
Summary:
Plumbs through an optional name for FollyScopedThreadScheduler.
This allows the threads to be named in XCode/lldb.

Differential Revision: D38582449

fbshipit-source-id: 1de50d25c0f91e7003cf81cb22faf4b10a8e23a8
2022-08-10 21:21:39 -07:00
facebook-flipper-bot
5788e8db84 Automated: Update Podfile.lock (#3944)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3944

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D38246079/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D38246079/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: mweststrate

Differential Revision: D38246079

Pulled By: passy

fbshipit-source-id: ac2adadf8fc83e6df3eccc7b93c524039f660cc4
2022-07-29 04:27:56 -07:00
Luke De Feo
f72406b06c Ensure that highlight always responds to prevent timeouts
Summary: In a previous diff D32278523 (8764da7c0b) The desktop request was changed from send to call. Call expects a response and not all code paths return a response. Most calls to set highlight are timing out.

Reviewed By: mweststrate

Differential Revision: D38074704

fbshipit-source-id: 6e85416d6b6470efaa177ad1b74420c8237366d5
2022-07-25 14:51:41 -07:00
John Knox
485083750e Fix NPE in ios sdk
Summary: Haven't been able to repro this, but from the stack trace it seems like this NSData object is null in some cases: https://www.internalfb.com/logview/details/instagram_ios_crashes/d0e929eae516ec48380518380cfca9f5/?selected-logview-tab=Employee+Traces

Reviewed By: lblasa

Differential Revision: D37995141

fbshipit-source-id: 937217a7a5b1c7cc992f3f9ca5dc3a7e31165d1c
2022-07-21 05:34:07 -07:00
Ruslan Latypov
88bf980b8e fix more imports
Summary:
There are many files across fbobjc relying on -include_pch and therefore they miss Foundation.h and UIKit.h includes. This diff was generated by a codemod and fixes these missing includes.
More details on the missing imports https://fb.workplace.com/groups/929548250966094/permalink/981237982463787/

Changelog: [Internal]

Reviewed By: yannickl

Differential Revision: D37282740

fbshipit-source-id: 0f419025b3cf2f811e96ff464cb19e8e5a25aa09
2022-07-19 12:53:11 -07:00
facebook-flipper-bot
41e7e2a338 Automated: Update Podfile.lock (#3885)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3885

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D37664209/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D37664209/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: mweststrate

Differential Revision: D37664209

Pulled By: passy

fbshipit-source-id: 6a69befdb36b27c21225560dde320867efc940f1
2022-07-07 06:15:47 -07:00
facebook-flipper-bot
091de0dd4c Automated: Update Podfile.lock (#3857)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3857

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D37481591/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D37481591/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: lblasa

Differential Revision: D37481591

Pulled By: nikoant

fbshipit-source-id: a2c6e3bd13b992122fce7e2ac467252258c0d5a5
2022-06-28 03:20:03 -07:00
Ruslan Latypov
e43884cf11 fixing more imports
Summary: Some files relying on -include_pch and therefore they miss Foundation.h and UIKit.h includes. This diff is fixing missing imports

Reviewed By: rmaz

Differential Revision: D37140239

fbshipit-source-id: bc57921e0c8365e0e9a5a571d607ba40ff1b31f3
2022-06-14 13:37:04 -07:00
facebook-flipper-bot
60f9a5e8de Automated: Update Podfile.lock (#3804)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3804

Reviewed By: nikoant

Differential Revision: D37068378

Pulled By: passy

fbshipit-source-id: d7f30af8f568f6440fc3ab66d4c1c1fe9a99790e
2022-06-10 05:40:58 -07:00
Yedidya Feldblum
48fec52610 cut config for folly::uncaught_exceptions
Summary: This function is no longer configured, so cut flipper's configuration of it.

Reviewed By: lblasa

Differential Revision: D34513641

fbshipit-source-id: 46e909a3a1ec7924ffd3b38ff7769020a486fda4
2022-05-28 16:47:31 -07:00
Lorenzo Blasa
e44cad5e99 Partially remove dependency on folly async
Summary:
This change isolates the usage of folly async from Flipper. Is now self-contained in Flipper Folly schedulers.

Users of Flipper can decide not to use the types defined in that header and implement their own.

NOTE: changes are minimal, we are just replacing direct calls to folly event base with a scheduler which simply relays this on to folly.

Reviewed By: fabiomassimo

Differential Revision: D36626483

fbshipit-source-id: add0241caf4af0aa5c3b5c2e7efc2e725f5400ab
2022-05-25 15:58:05 -07:00
facebook-flipper-bot
77ada56e02 Automated: Update Podfile.lock (#3743)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3743

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D36593800/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D36593800/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: passy

Differential Revision: D36593800

Pulled By: nikoant

fbshipit-source-id: c08c622c19624ad6005919eef8e30c2a4f78da7f
2022-05-23 08:04:43 -07:00
facebook-flipper-bot
2704ce5a8e Automated: Update Podfile.lock (#3736)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3736

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D36513699/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D36513699/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: passy

Differential Revision: D36513699

Pulled By: nikoant

fbshipit-source-id: 96442180142366984e0fcff8926cf3870f79197a
2022-05-19 07:16:08 -07:00
Billy Ng
3804ccf898 Revert D36052198: Partially remove dependency on folly async
Differential Revision:
D36052198 (ade685c621)

Original commit changeset: 170d64a324a1

Original Phabricator Diff: D36052198 (ade685c621)

fbshipit-source-id: 69d2b18e70a6267667432d6ed9dc1c5bc545b417
2022-05-12 18:47:41 -07:00
Lorenzo Blasa
ade685c621 Partially remove dependency on folly async
Summary:
This change isolates the usage of folly async from Flipper. Is now self-contained in Flipper Folly schedulers.

Users of Flipper can decide not to use the types defined in that header and implement their own.

NOTE: changes are minimal, we are just replacing direct calls to folly event base with a scheduler which simply relays this on to folly.

Reviewed By: fabiomassimo

Differential Revision: D36052198

fbshipit-source-id: 170d64a324a1f1f100224e2622a59cbac3c8b642
2022-05-12 17:56:17 -07:00
Lorenzo Blasa
63dde6e5cf Exceptions
Summary:
Flipper doesn really use library specific exceptions throughout, and that's OK.

Introducing SSLException as a replacement for the existing Folly Async Socket SSL exception.

This exception originally thrown by rsocket. Because we had rsocket and websockets using the same code, websockets were creating and throwing this same exception.

With rsocket gone, we can fully replace the usage of that exception. This is also needed as to decouple Flipper from folly async components.

Reviewed By: fabiomassimo

Differential Revision: D36245624

fbshipit-source-id: f5c97c5efe063280ce95be130008dee7f4e5d788
2022-05-12 09:16:13 -07:00
Lorenzo Blasa
3826f2c8ef Refactor endpoint verification prior to connection
Summary:
^

There's a similar issue/request for Android. So, this change moves the code out from the FlipperKit into Flipper as to be able to reuse it.

Reviewed By: aigoncharov

Differential Revision: D35961745

fbshipit-source-id: aa255db582a7852dc06c2feaba389d1dac3b0f67
2022-05-03 11:24:17 -07:00
Lorenzo Blasa
7d392ade29 Diagnostics UI (Dark mode)
Summary:
^
In dark mode, the logs were not visible as font color is white, same as the background color of the label explicitly set to white.

Don't set it to white (unless the text color is changed as well).

Reviewed By: antonk52

Differential Revision: D36015027

fbshipit-source-id: bf37546a13e4a2153f7307ddeebddb23f27f4403
2022-04-28 14:42:50 -07:00
facebook-flipper-bot
836fc06523 Automated: Update Podfile.lock (#3666)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3666

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D35969376/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D35969376/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: nikoant

Differential Revision: D35969376

Pulled By: passy

fbshipit-source-id: 1ef9ae049307d1a0a9be123a9e0215c6fc22cde9
2022-04-28 07:59:56 -07:00
facebook-flipper-bot
5362649432 Automated: Update Podfile.lock (#3647)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3647

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D35810508/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D35810508/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: nikoant

Differential Revision: D35810508

Pulled By: lblasa

fbshipit-source-id: 5087f032d7982cbc03f03e2b1619343f3123ae71
2022-04-21 03:49:13 -07:00
Lorenzo Blasa
f1fe66afd9 Use a dispatch queue for websocket operations
Summary:
This diff ensures that all operations on the socket are put into a serial background queue, including delegate callbacks.

All operations are executed asynchronously except disconnect, which is made synchronous as to guarantee no resources are accessed after the call.

Reviewed By: fabiomassimo

Differential Revision: D35254499

fbshipit-source-id: 33d93926f7bfc8948095c59f12ca31f0a932b8ae
2022-04-12 02:30:02 -07:00
Lorenzo Blasa
30a1d09a83 Dealloc safety improvement
Summary:
From a different diff, it was pointed out that calling methods on self are discouraged:

Generally you want to avoid calling methods on self in dealloc because it allows you to accidentally capture references to self in dealloc, which leads to zombies.

In this case, a disconnect effectively invalidates a timer and disconnect the underlying socket. Both take place either way when the members are deallocated too. The only thing is not done is notify the caller via the event handler of a close event.

In our case, the caller actually manually disconnects before deallocating so this is not an issue.

Reviewed By: fabiomassimo

Differential Revision: D35547600

fbshipit-source-id: 9b5b9892b657a69585943613bc81344b38dbef30
2022-04-12 02:30:02 -07:00
Lorenzo Blasa
fd2d7bc1a4 Attempt to socket connect before establishing websocket connection
Summary:
^

Changelog: Check if there's a process listening at the specified port before attempting to establish a websocket connection on iOS

Reviewed By: fabiomassimo

Differential Revision: D35546817

fbshipit-source-id: 92ccca9afd8bcdc6d79205cc277ac813e0999166
2022-04-11 07:03:51 -07:00
facebook-flipper-bot
9cf630b27b Automated: Update Podfile.lock (#3615)
Summary:
This is an automated PR to update the Podfile.lock.
- Make sure that the Podfile.lock contains latest FlipperKit and Flipper pod versions.
- Also make sure that all the dependencies are updated to the latest one.
- This is auto-generated by [create-pull-request](https://github.com/peter-evans/create-pull-request)

Pull Request resolved: https://github.com/facebook/flipper/pull/3615

Test Plan:
Imported from GitHub, without a `Test Plan:` line.

**Static Docs Preview: flipper**
|[Full Site](https://our.intern.facebook.com/intern/staticdocs/eph/D35435008/V2/flipper/)|

|**Modified Pages**|
|[docs/getting-started/ios-native](https://our.intern.facebook.com/intern/staticdocs/eph/D35435008/V2/flipper/docs/getting-started/ios-native/)|

Reviewed By: passy

Differential Revision: D35435008

Pulled By: nikoant

fbshipit-source-id: 7913265e6c15105f167c2ca3257242030909d4fe
2022-04-07 03:40:19 -07:00