You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix(providers): prevent ReferenceError: validationPsd is not defined for keyless providers - #13064
For keyless providers (such as searxng-search), formData.apiKey is empty. validationPsd was previously scoped inside the if (!isOAuth && formData.apiKey) block, but referenced later in the scope when assigning updates.providerSpecificData.
Moving let validationPsd = validatedProviderSpecificData; out of the if (!isOAuth && formData.apiKey) block ensures validationPsd is always in scope when saving keyless provider connections.
Thanks for the fix and for rebasing onto release/v3.8.51! I checked the current tip and let validationPsd = validatedProviderSpecificData; is already present at that exact spot
(it traces back to #8949, merged before your branch point) — so this PR's one-line change is
already on the tip and doesn't change behavior. Could you confirm whether the original ReferenceError from #13063 still reproduces on current release/v3.8.51? If it doesn't, I'll
close this as already fixed; if it does, there's likely a different code path causing it and
I'd like to see the exact repro steps.
Triage note: this is the review recommendation — the close itself happens only after the maintainer's per-PR sign-off (and, where a superseding PR is named, after it has landed). Nothing is being closed by this comment.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #13063
Summary
For keyless providers (such as
searxng-search),formData.apiKeyis empty.validationPsdwas previously scoped inside theif (!isOAuth && formData.apiKey)block, but referenced later in the scope when assigningupdates.providerSpecificData.Moving
let validationPsd = validatedProviderSpecificData;out of theif (!isOAuth && formData.apiKey)block ensuresvalidationPsdis always in scope when saving keyless provider connections.