-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
cleanup http config in legacy paltform for migrated params #42818
cleanup http config in legacy paltform for migrated params #42818
Conversation
6d55242
to
d6820fa
Compare
💔 Build Failed |
d6820fa
to
ed5b2ba
Compare
ed5b2ba
to
393d1ea
Compare
💔 Build Failed |
💚 Build Succeeded |
Pinging @elastic/kibana-platform |
💚 Build Succeeded |
xsrf: Joi.object({ | ||
disableProtection: Joi.boolean().default(false), | ||
whitelist: Joi.array().items( | ||
Joi.string().regex(/^\//, 'start with a slash') | ||
).default([]), | ||
token: Joi.string().optional().notes('Deprecated') | ||
}).default(), | ||
|
||
// keep them for BWC, remove when not used in Legacy. | ||
// validation should be in sync with one in New platform. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a file path for where these should be in sync?
Can you also add a comment in the corresponding NP code (unless there already is one)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a file path for where these should be in sync?
ok. it's not always practical, because a file can be moved and comments are out of sync
Can you also add a comment in the corresponding NP code (unless there already is one)?
will do
@@ -101,128 +101,6 @@ describe('Config schema', function () { | |||
}); | |||
}); | |||
|
|||
describe('ssl', function () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume parallel tests exist in the NP already?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💚 Build Succeeded |
…2818) * cleanup http config in legacy paltform for migrated params * add comments
…2818) * cleanup http config in legacy paltform for migrated params * add comments
Summary
Closes #41740
Remove validation rules for config parameters handled in the New platform.
blocker for #42606
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.[ ] This was checked for cross-browser compatibility, including a check against IE11[ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes i18n support[ ] Documentation was added for features that require explanation or tutorials[ ] This was checked for keyboard-only and screenreader accessibilityFor maintainers