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

feat(flags): Add LaunchDarkly integration #14207

Open
wants to merge 62 commits into
base: develop
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
62 commits
Select commit Hold shift + click to select a range
26722d7
Init package files
aliu39 Oct 29, 2024
72f9cc0
Merge branch 'develop' into aliu/launch-darkly
aliu39 Oct 29, 2024
419cd83
Revert changelog and move types file
aliu39 Oct 29, 2024
610da4d
Add ld to dependencies and skeleton code. Get rid of core/
aliu39 Oct 29, 2024
5b2e5ec
Fix readme, rename types file, bring back core/
aliu39 Oct 29, 2024
66b1253
Fix readme 2
aliu39 Oct 30, 2024
023604a
Merge branch 'develop' into aliu/launch-darkly
aliu39 Oct 30, 2024
1f99c28
Merge branch 'aliu/launch-darkly' of https://github.com/getsentry/sen…
aliu39 Oct 30, 2024
85acc6d
Finish implementing, minus scope.flags
aliu39 Oct 30, 2024
3b3a767
Implement flag buffer in sentry scope
aliu39 Oct 30, 2024
ab3181d
Revert changelog
aliu39 Oct 30, 2024
22684ea
fix types
michellewzhang Oct 30, 2024
79e5b24
docstring
michellewzhang Oct 30, 2024
63649c5
Export FeatureFlag type in index.ts
aliu39 Oct 30, 2024
923500f
Merge branch 'aliu/launch-darkly' of https://github.com/getsentry/sen…
aliu39 Oct 30, 2024
bd04755
Clean up comments
aliu39 Oct 30, 2024
91a4db9
Fix build (uses yalc for scope changes) and use LRUMap util
aliu39 Oct 31, 2024
1849673
Merge branch 'develop' into aliu/launch-darkly
aliu39 Nov 5, 2024
1ca9d53
Tweak hook name and docs
aliu39 Nov 5, 2024
893cd62
FlagBuffer class and interface, add to scope._contexts. Remove old fl…
aliu39 Nov 6, 2024
f172683
Call clone in scope.clone()
aliu39 Nov 6, 2024
b96c17f
Rewrite as a util fx instead of class. Use in LD integration
aliu39 Nov 6, 2024
f9a659b
Delete LD pkg
aliu39 Nov 6, 2024
d9c5860
Delete LD from package.json and update lock
aliu39 Nov 6, 2024
cf24b75
Readd LD integration
aliu39 Nov 6, 2024
5561633
Fix dependencies and rename exports
aliu39 Nov 7, 2024
6eab55f
Fix normalize depth of 3 for contexts.flags, improve docstrings
aliu39 Nov 8, 2024
d90d346
Rm eslint rules, todos
aliu39 Nov 8, 2024
19ccb19
Merge branch 'develop' into aliu/launch-darkly-integration
aliu39 Nov 8, 2024
548858d
Update sentry dependency vers
aliu39 Nov 9, 2024
c289e63
Update insertToFlagBuffer and unit test it
aliu39 Nov 9, 2024
f5c6995
Move ld to devDependencies and rm shim-preact from build:types
aliu39 Nov 11, 2024
334aa51
Move to browser integrations, delete package
aliu39 Nov 11, 2024
c27f766
Init browser-integration-tests
aliu39 Nov 12, 2024
948cea7
Fix test
aliu39 Nov 12, 2024
d371316
Merge branch 'develop' into aliu/launch-darkly-integration
aliu39 Nov 12, 2024
aef6dba
Finialize test and make handler synchronous
aliu39 Nov 12, 2024
7b14926
Move util to browser pkg
aliu39 Nov 12, 2024
8ec8680
Review comments, fix buffer size import, clean up todos
aliu39 Nov 13, 2024
6e22c17
Restructure src and test modules, rename util and type files
aliu39 Nov 13, 2024
dc7e74a
Inline LD types
aliu39 Nov 13, 2024
3c686c5
Copy flags when attaching to event
aliu39 Nov 13, 2024
88f8251
Add withScope test and delete bad not modified test
aliu39 Nov 13, 2024
05036ae
Merge branch 'develop' into aliu/launch-darkly-integration
aliu39 Nov 13, 2024
fc79f4a
Del unnecesary devDep
aliu39 Nov 13, 2024
7a5f29d
Log insertToFlagBuffer error instead of throwing
aliu39 Nov 13, 2024
7e24158
Merge branch 'develop' into aliu/launch-darkly-integration
aliu39 Nov 13, 2024
969c5c0
Fix playwright tests
aliu39 Nov 13, 2024
ac8faf7
Use Sentry namespace in init
aliu39 Nov 13, 2024
b1cb0a5
(Temporarily?) move to exports
aliu39 Nov 13, 2024
3b784a6
Skip CDN bundle tests, fix unit test, un-export FLAG_BUFFER_SIZE, upd…
aliu39 Nov 13, 2024
b93c062
Merge branch 'develop' into aliu/launch-darkly-integration
aliu39 Nov 13, 2024
8ffafd1
Revert lockfile
aliu39 Nov 13, 2024
83ee34b
biome formatting
aliu39 Nov 13, 2024
a2ba257
Bump size limits for vue (w/tracing) and browser (w/async feedback)
aliu39 Nov 14, 2024
611df2c
Merge branch 'develop' into aliu/launch-darkly-integration
aliu39 Nov 14, 2024
be9ccc2
Merge branch 'develop' into aliu/launch-darkly-integration
billyvg Nov 19, 2024
7b3382f
make `shouldSkipLaunchDarklyTest` to be generic for feature flags
billyvg Nov 19, 2024
d292da2
Merge branch 'develop' into aliu/launch-darkly-integration
billyvg Nov 20, 2024
c4ba65b
fix tests
billyvg Nov 19, 2024
fb0bf05
Update dev-packages/browser-integration-tests/utils/helpers.ts
billyvg Nov 20, 2024
90577bf
import from core
billyvg Nov 20, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion packages/browser/src/utils/featureFlags.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
import type { FeatureFlag } from '@sentry/types';
import { logger } from '@sentry/utils';
import { DEBUG_BUILD } from '../debug-build';

/**
* Ordered LRU cache for storing feature flags in the scope context. The name
Expand Down Expand Up @@ -32,7 +34,8 @@ export function insertToFlagBuffer(
maxSize: number = FLAG_BUFFER_SIZE,
): void {
if (flags.length > maxSize) {
throw Error(`insertToFlagBuffer called on a buffer larger than the given maxSize=${maxSize}`);
DEBUG_BUILD && logger.error(`insertToFlagBuffer called on a buffer larger than the given maxSize=${maxSize}`);
aliu39 marked this conversation as resolved.
Show resolved Hide resolved
return;
}

// Check if the flag is already in the buffer - O(n)
Expand Down