Sidebranch: Transform Secret Engine Initial setup#9625
Merged
Conversation
…rm/role adapter and model.
andaley
reviewed
Jul 30, 2020
| store: service(), | ||
| actions: { | ||
| // TODO modify the parameters and potentially rename | ||
| createOrUpdate(type, event) { |
Contributor
There was a problem hiding this comment.
you could rename this to handleCreateOrUpdate if you want to be clear that this action is an event handler calling the method on the adapter. in ember we see a lot of handleFoo where foo is either the event (submit, enter) or an action passed down from above.
andaley
reviewed
Jul 30, 2020
andaley
approved these changes
Jul 30, 2020
chelshaw
added a commit
that referenced
this pull request
Aug 26, 2020
* Ui/transform enable (#9647) * Show Transform on engines list if enterprise * Add box-radio component * Add is-disabled styling for box-radio and fix tooltip styling when position: above * Add KMIP and Transform to possible features on has feature helper * Sidebranch: Transform Secret Engine Initial setup (#9625) * WIP // list transforms, console.logs and all * setup LIST transformations ajax request and draft out options-for-backend options * change from plural to singluar and add transform to secret-edit * create two transform edit components * modify transform model with new attrs * add adapterFor to connect transform adapter to transform-edit-form component * setup Allowed roles searchSelect component to search over new transform/role adapter and model. * clean up for PR * clean up linting errors * restructure adapter call, now it works. * remove console * setup template model for SearchSelect component * add props to form field and search select for styling Co-authored-by: Chelsea Shaw <chelshaw.dev@gmail.com> * Ui/transform language fixes (#9666) * Update casing and wording on Transform list route. Use generic list item for transformations * Add back js file for transformation-edit * Set up transform for tabs * Ui/create edit transformation fixes (#9668) * add conditional for masking vs tweak source based on type, and update text for create transformation * change order * fix error with stringArray * setup the edit/delete transformation view * clean up toolbar links * setup serializer to change response of mask character from keycode to character * change styling of label and sub-text size, confirmed with design * temp fix on templates vs template * add clickable list item * add space between template list * setup styling and structure for the rest of the show transformation. TODO: turn into components. * create transform-show-transformation component * add attachCapabilities to transform model and update transform-transformation-itme list accordingly * clean up liniting errors * address pr comments * remove leftover * clean up * Sidebranch: UI transform create and edit clean up (#9778) * clean up some of the TODOs * setup edit view with read only attributes for name and template * setup initial selected for search select component * fixes * hide templates form field for now * set selectLimit for search select component * hide power select if the select limit is greater than or equal to the selectedOptions length * clean up failing linting * address pr comments * Ui/fix list roles transformation (#9788) * Update search-select to pass backend to query if exists * Update role and template adapters * cleanup * Fix replace with static string * Ui/transform cleanup 2 (#9789) * amend encode/decode commands for now until design gets back with more details * restrict character count on masking input field * clean up selectLimit * show backend instead of transform in cli copy command * Show KMIP un-selectable if enterprise but no ADP module (#9780) * New component transform-edit-base * Duplicate RoleEdit as TransformEditBase and swap in all transform components * Roll back role-edit changes * Update to transform edit base * Remove extraeneous set backend type on transform components * formatting * Revert search-select changes * Update template/templates data on transformation (#9838) Co-authored-by: Angel Garbarino <Monkeychip@users.noreply.github.com>
sapk
pushed a commit
to sapk-fork/vault
that referenced
this pull request
Oct 6, 2025
…rp#9699) (hashicorp#9711) * Copy non-backport labels during backport PR * whitespace * Debug logs * update to use libgithub.Label * Update tools/pipeline/internal/pkg/github/create_backport_test.go --------- Co-authored-by: Luis (LT) Carbonell <lt.carbonell@hashicorp.com> Co-authored-by: Ryan Cragun <me@ryan.ec>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Chelsea and I have paired through the initial setup of the Transform Secret Engine. This PR get's us started. We are starting with a sidebranch until the initial setup is ready to push towards master.
In this PR we do the following:
Setup the Transform Adapter and Model so that you can navigate to the first page --listing and then creating a Transformation.
Call
createOrUpdatefrom the component and bubble it up to the Transform adapter. We were able to do this following similar patterns for PKI, where we extended the EditRole component. This is slightly different than how the SSH engine handles requests. We wanted to stay away from partials and instead use nested components.Create the transform/role model and adapter to populate the "Allowed Roles"
searchSelectcomponents model attr. This component searches over a model to populate what a user can search. The transform/role adapter makes a request to return a list of roles and returns it to the transform/model.In the transform model, define the attrs of each type of form field so that when the
formFieldcomponent loops through it know how to display each type. This is slightly different than how the SSH Secret Engine works, again because we wanted to avoid partials and have opted to not take advantage of openAPI. Currently, openAPI does not give us enough flexibility to match the designs of this project. We may come back after the fact add addresses this, but for now, we are moving forward without relying on openAPI to design the forms.Here's a gif of the work so far.
Another gif where we can create a Transformation (updated after we figured out the correct adapter/serializer pattern)
Here is an image with the
subTextandsubLabelprops.