Skip to content

Commit 9525f11

Browse files
committed
fix(context): do not force-default routingStrategy to 'query'
Leave routingStrategy undefined when not explicitly set — only warn. The path adapter already handles the undefined case as a fallback, so forcing 'query' would be a behavior change for existing apps that never declared a strategy.
1 parent 217f5f7 commit 9525f11

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

packages/modules/context/src/configurator.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -257,9 +257,8 @@ export class ContextModuleConfigurator implements IContextModuleConfigurator {
257257
if (!config.routingStrategy) {
258258
console.warn(
259259
'ContextModuleConfigurator.createConfig',
260-
"missing routing strategy, defaulting to 'query' (recommended)",
260+
"missing routing strategy — declare setRoutingStrategy('query') or setRoutingStrategy('path') explicitly",
261261
);
262-
config.routingStrategy = 'query';
263262
}
264263

265264
config.resolveInitialContext ??= resolveInitialContext();

0 commit comments

Comments
 (0)