PostHog / posthog-js-lite

Reimplementation of posthog-js to be as light and modular as possible.
https://posthog.com/docs/libraries
MIT License
70 stars 36 forks source link

fix: Enforce the shutdown timeout #305

Closed benjackwhite closed 1 week ago

benjackwhite commented 2 weeks ago

Problem

The shutdown timeout is misleading. If a fetch request is timing out we may still sit on it as we only do the check after it is done.

Changes

Change to a timeout based solution. It's a little complex but i think thats just the way.

Release info Sub-libraries affected

Bump level

Libraries affected

Changelog notes

github-actions[bot] commented 2 weeks ago

Size Change: +751 B (+0.71%)

Total Size: 107 kB

Filename Size Change
posthog-node/lib/index.cjs.js 20.2 kB +153 B (+0.76%)
posthog-node/lib/index.esm.js 20.1 kB +153 B (+0.77%)
posthog-react-native/lib/posthog-core/src/index.js 9.86 kB +137 B (+1.41%)
posthog-web/lib/index.cjs.js 17.3 kB +154 B (+0.9%)
posthog-web/lib/index.esm.js 17.2 kB +154 B (+0.9%)
ℹ️ View Unchanged | Filename | Size | | :--- | :---: | | `posthog-react-native/lib/posthog-core/src/eventemitter.js` | 1.08 kB | | `posthog-react-native/lib/posthog-core/src/lz-string.js` | 1.42 kB | | `posthog-react-native/lib/posthog-core/src/types.js` | 365 B | | `posthog-react-native/lib/posthog-core/src/utils.js` | 822 B | | `posthog-react-native/lib/posthog-core/src/vendor/uuidv7.js` | 2.04 kB | | `posthog-react-native/lib/posthog-react-native/index.js` | 485 B | | `posthog-react-native/lib/posthog-react-native/src/autocapture.js` | 1.8 kB | | `posthog-react-native/lib/posthog-react-native/src/frameworks/wix-navigation.js` | 651 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useFeatureFlag.js` | 437 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useFeatureFlags.js` | 362 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useLifecycleTracker.js` | 416 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/useNavigationTracker.js` | 628 B | | `posthog-react-native/lib/posthog-react-native/src/hooks/usePostHog.js` | 249 B | | `posthog-react-native/lib/posthog-react-native/src/legacy.js` | 810 B | | `posthog-react-native/lib/posthog-react-native/src/native-deps.js` | 1.17 kB | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalAsyncStorage.js` | 183 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoApplication.js` | 215 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoDevice.js` | 211 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoFileSystem.js` | 224 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalExpoLocalization.js` | 216 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeDeviceInfo.js` | 220 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeNavigation.js` | 218 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalReactNativeNavigationWix.js` | 222 B | | `posthog-react-native/lib/posthog-react-native/src/optional/OptionalSessionReplay.js` | 231 B | | `posthog-react-native/lib/posthog-react-native/src/posthog-rn.js` | 4.08 kB | | `posthog-react-native/lib/posthog-react-native/src/PostHogContext.js` | 210 B | | `posthog-react-native/lib/posthog-react-native/src/PostHogProvider.js` | 1.57 kB | | `posthog-react-native/lib/posthog-react-native/src/storage.js` | 1.09 kB | | `posthog-react-native/lib/posthog-react-native/src/types.js` | 90 B | | `posthog-react-native/lib/posthog-react-native/src/version.js` | 124 B |

compressed-size-action

marandaneto commented 2 weeks ago

@benjackwhite if you add changelog - update version we can cut a version o/