consul

Commit Graph

Author	SHA1	Message	Date
John Cowen	8192dde485	ui: Unskip auth-method serializer test (#10878 ) During #9617 we added a list view only for AuthMethods, but not a detail view. We did add the Adapter/Serializer that collected/reshaped data for a detail view. The test for this serializer was skipped here, but I'm not sure why. We then added #9845 which began to use this AuthMethod Serializer, but we didn't go back to finish up the skipped test here either. This PR unskips this test and finishes off the test correctly.	3 years ago
John Cowen	05a28c3111	ui: [BUGFIX] Properly encode non-URL safe characters in OIDC responses (#10901 ) This commit fixes 2 problems with our OIDC flow in the UI, the first is straightforwards, the second is relatively more in depth: 1: A typo (1.10.1 only) During #10503 we injected our settings service into the our oidc-provider service, there are some comments in the PR as to the whys and wherefores for this change (https://github.com/hashicorp/consul/pull/10503/files#diff-aa2ffda6d0a966ba631c079fa3a5f60a2a1bdc7eed5b3a98ee7b5b682f1cb4c3R28) Fixing the typo so it was no longer looking for an unknown service (repository/settings > settings) fixed this. 2: URL encoding (1.9.x, 1.10.x) TL;DR: /oidc/authorize/provider/with/slashes/code/with/slashes/status/with/slashes should be /oidc/authorize/provider%2Fwith%2Fslashes/code%2Fwith%2Fslashes/status%2Fwith%2Fslashes When we receive our authorization response back from the OIDC 3rd party, we POST the code and status data from that response back to consul via acallback as part of the OIDC flow. From what I remember back when this feature was originally added, the method is a POST request to avoid folks putting secret-like things into API requests/URLs/query params that are more likely to be visible to the human eye, and POSTing is expected behaviour. Additionally, in the UI we identify all external resources using unique resource identifiers. Our OIDC flow uses these resources and their identifiers to perform the OIDC flow using a declarative state machine. If any information in these identifiers uses non-URL-safe characters then these characters require URL encoding and we added a helper a while back to specifically help us to do this once we started using this for things that required URL encoding. The final fix here make sure that we URL encode code and status before using them with one of our unique resource identifiers, just like we do with the majority of other places where we use these identifiers.	3 years ago
Kenia	ae9c313382	ui: Update intention permissions notice wording (#10836 )	3 years ago
Kenia	019ce785ab	ui: Create Routing Configurations route and page (#10835 )	3 years ago
Kenia	ab6a675209	ui: Split up the socket mode from the socket path (#10581 )	3 years ago
Kenia	bcd53e73a2	ui: Add Vault as a Service External Source (#10769 )	3 years ago
Kenia	8ad1ab9c08	ui: Fix Health Checks in K/V form Lock Sessions Info section (#10767 )	3 years ago
Kenia	2ee501be8d	ui: Add copy button for Secret ID in Tokens list page (#10735 )	3 years ago
Kenia	eb5512fb74	ui: Fix dropdown option duplications (#10706 )	3 years ago
Kenia	1eb7a83eec	ui: Add tests for topology metrics stats (#10600 )	3 years ago
John Cowen	0762da3a62	ui: [BUGFIX] Ensure we use the ns query param name when requesting permissions (#10608 ) Previously when namespaces were enabled, we weren't requesting permission for the actively selected namespace, and instead always checking the permissions for the default namespace. This commit ensures we request permissions for the actively selected namespace.	3 years ago
John Cowen	3e80e637ba	ui: [BUGFIX] Fix KV Code Editor syntax loading (#10605 ) This commit adds a bit of string wrangling to avoid the keys in our javascript source file also being transformed. Additionally, whilst looking at this we decided that Maps are a better dictionary than javascript objects, so we moved to use those here also (but this doesn't affect the issue)	3 years ago
John Cowen	54f0cd812a	ui: Show the correct 'ACLs Disabled' page when ACLs are disabled (#10604 ) Adds 'can access ACLs' which means one of two things 1. When ACLs are disabled I can access the 'please enable ACLs' page 2. When ACLs are enabled, its the same as canRead	3 years ago
John Cowen	b256313256	ui: [BUGFIX] Ensure in-folder KVs are created in the correct folder (#10569 ) When clicking to create a KV within folder name, would would be viewing a form that was a form for creating a KV in the root, which when the user clicked to save, saved the KV in the root. For the moment at least I've removed the code that strips double slashes, and whilst this isn't ideal, it looks like we've picked up one of those bugs that turns into a 'feature', and completely reworking KV to not rely on the double slashes is not really an option right now.	3 years ago
Kenia	a96e87aece	ui: Add socket icon for UDS (#10573 )	3 years ago
John Cowen	6fbeea5def	ui: Don't default to the default namespace, use the token default namespace instead (#10503 ) The default namespace, and the tokens default namespace (or its origin namespace) is slightly more complicated than other things we deal with in the UI, there's plenty of info/docs on this that I've added in this PR. Previously: When a namespace was not specified in the URL, we used to default to the default namespace. When you logged in using a token we automatically forward you the namespace URL that your token originates from, so you are then using the namespace for your token by default. You can of course then edit the URL to remove the namespace portion, or perhaps revisit the UI at the root path with you token already set. In these latter cases we would show you information from the default namespace. So if you had no namespace segment/portion in the URL, we would assume default, perform actions against the default namespace and highlight the default namespace in the namespace selector menu. If you wanted to perform actions in your tokens origin namespace you would have to manually select it from the namespace selector menu. This PR: Now, when you have no namespace segment/portion in the URL, we use the token's origin namespace instead (and if you don't have a token, we then use the default namespace like it was previously) Notes/thoughts: I originally thought we were showing an incorrectly selected namespace in the namespace selector, but it also matched up with what we were doing with the API, so it was in fact correct. The issue was more that we weren't selecting the origin namespace of the token for the user when a namespace segment was omitted from the URL. Seeing as we automatically forward you to the tokens origin namespace when you log in, and we were correctly showing the namespace we were acting on when you had no namespace segment in the URL (in the previous case default), I'm not entirely sure how much of an issue this actually was. This characteristic of namespace+token+namespace is a little weird and its easy to miss a subtlety or two so I tried to add some documentation in here for future me/someone else (including some in depth code comment around one of the API endpoints where this is very subtle and very hard to miss). I'm not the greatest at words, so would be great to get some edits there if it doesn't seem clear to folks. The fact that we used to save your previous datacenter and namespace into local storage for reasons also meant the interaction here was slightly more complicated than it needed to be, so whilst we were here we rejigged things slightly to satisfy said reasons still but not use local storage (we try and grab the info from higher up). A lot of the related code here is from before we had our Routlets which I think could probably make all of this a lot less complicated, but I didn't want to do a wholesale replacement in this PR, we can save that for a separate PR on its own at some point.	3 years ago
John Cowen	74070c0955	ui: Add intl debug helpers (#10513 ) This commit adds a couple of debug utilities to help us to continue slowly adding i18n support: - We've added a CONSUL_INTL_DEBUG env/cookie variable to turn off variable interpolation within the t helper so you can see which variables are being interpolated. - We've added a CONSUL_INTL_LOCALE env/cookie which currently supports two 'pseudo-locales' - la-fk (fake latin) and - (just dashes) either of which will make it easier to see what has not been localized until we can add prettier rules to prevent adding any copy into templates at all. I would guess if we ever translated the app we would use this for looking at things whilst developing also - but as yet I've not adding anything for that here seeing as we don't translate anything. Both variables are dev-time only and all code for this is removed from the production build.	3 years ago
John Cowen	b0d69efcc1	ui: Allow disabling of sourcemaps via env var (#10491 )	3 years ago
John Cowen	e541a43b37	ui: Fixup definition-table + copy-button margin (#10512 )	3 years ago
John Cowen	a6996b6ea5	ui: CopyButton amends (#10511 ) * ui: Add with-copyable modifier * Use with-copyable modifier for our own CopyButton * Move copy-button styling and remove most of `copy-btn`	3 years ago
Daniel Nephin	3a045cca8d	ca: remove unused RotationPeriod field This field was never used. Since it is persisted as part of a map[string]interface{} it is pretty easy to remove it.	3 years ago
Jared Kirschner	bd536151e1	Replace use of 'sane' where appropriate HashiCorp voice, style, and language guidelines recommend avoiding ableist language unless its reference to ability is accurate in a particular use.	3 years ago
John Cowen	ace794d214	ui: Enable specifying additional docfy config as json (#10464 )	3 years ago
John Cowen	c78f7ecb27	ui: Colocate remaining CSS components to the app/components folder (#10328 )	3 years ago
John Cowen	043f631b7f	ui: Rename icons for consistency and remove unused icons (#10311 ) * ui: Standardize logo naming According to structure it should always be logo-name not name-logo * Make sure all our logos use logo-name format * Upgrade to @hashicorp/structure-icons 1.9.0 * Add `-color` to be consistent with other logos * Add ms logo back in * Remove all the old `-color` icons from before when we got masks Add missing files * Missed glimmer extend name change	3 years ago
John Cowen	4bddd52103	ui: Increase z-index of main menus to avoid layering issues (#10428 )	3 years ago
John Cowen	5f17062b0a	ui: Add resolution for xmlhttprequest-ssl at least 1.6.3 (#10370 )	3 years ago
John Cowen	9eab715149	ui: Move all our icons to use CSS custom properties instead of SASS vars (#10298 ) * ui: Move all our icons to use CSS custom properties The good thing about SASS vars is, if you don't use them they get removed from the final build. Whereas with CSS we have no tree-shaking to get rid of unused CSS custom properties. We can mostly work around this and for some things like colors its no big deal if we have some hex-codes in the build that we don't use as hex-codes are relatively small. We've been slowly but surely moving all of our colors (and other things) to use CSS custom properties instead of SASS vars now that we have them available. This commit makes use of the 'tree-shaking' abilities of @extend to ensure that we only compile in the icons that we use. This commit is mostly churn-less as we already use @extend for the majority of our icons, so generally there is zero change here for working on the UI, but I did spot one single place where we were using SASS vars instead of @extend. This now uses the new form (second commit) Interestingly this reduces our CSS payload by ~2kb to ~53kb (around 25kb of that is these icons)	3 years ago
John Cowen	ac424187fe	ui: %horizontal-kv-list CSS component (and related) (#10285 ) This commit uses docfy to isolate the individual parts and options and investigates the why you might use certain options and document how you might use certain options. Originally we used a single %icon-definition CSS component to represent this, but seeing as some of them don't have icons, it didn't seem like the best name. So this PR splits this component into various different ones and then uses the new ones to continue to provide a now deprecated %icon-definition. The component is currently a CSS only component that assumes a single (or multiple) description lists for its markup component, and provides for multiple different options (including a reversed mode which I'm still not totally sure about, but we don't use this right now anyway). - %icon-definition - %horizontal-kv-list - %csv-list - %tag-list - %badge	3 years ago
John Cowen	18190fb071	ui: Move intention description field (#10292 ) ...from under the permissions widget to over the permissions widget	4 years ago
John Cowen	ddee7afbb9	ui: Fixup prettier for scss files and run (#10296 ) * ui: Ensure prettier makes all our scss files pretty * Run prettier on all our scss files	4 years ago
John Cowen	af72b9e9ab	ui: Unix Domain Socket support (#10287 ) This commit adds UI support for Unix Domain Sockets for upstream and downstreams (see #9981 and #10252)	4 years ago
John Cowen	2e4c9f5330	ui: Support Route optional parameters/segments (#10212 ) Moves our URLs with 'optional namespace segment' into a separately abstracted 'optional URL segment' feature	4 years ago
John Cowen	e11454cf71	ui: Colocate old base components into our app/component folder (#10275 )	4 years ago
John Cowen	a73b38a368	ui: Minor CSS tweaks (#10295 )	4 years ago
Kenia	f3efde3843	ui: Create and use collapsible notices component (#10270 ) * Create and use collapsible notices * Refactor collapsible-notices * Split up the topology acceptance tests * Add acceptance tests for tproxy notices * Add component file * Adds additional TProxy notices tests * Adds conditional to only show collapsable if more than 2 notices are present * Adds changelog * Refactorting the conditonal for collapsing the notices * Renaming undefinedIntention to be notDefinedIntention * Refactor tests	4 years ago
John Cowen	600f857531	ui: Add docs for AppView (#10265 ) * ui: change coloring of secondary navigation elements * Remove top border, this was probably from older designs/iterations * ui: Move app-view styles into components also... 1. Remove dead %app-view-content-error 2. Remove TabNav border overwriting * Bring into line with our 'project standard' class/attributes pattern * Add docs for AppView	4 years ago
John Cowen	dd4a66808d	ui: change coloring of secondary navigation elements (#10259 )	4 years ago
John Cowen	4e420df7ae	ui: Adds QUnit toggle bookmarklet to our docs (#10269 )	4 years ago
John Cowen	39302041e9	ui: Miscellaneous Lock Session fixes (#10225 )	4 years ago
John Cowen	8301e79c56	ui: Bump node to v14 (#10238 )	4 years ago
John Cowen	d1bbe0895f	ui: Use babel plugin to strip calls to runInDebug (#10237 )	4 years ago
John Cowen	04bd576179	ui: Serf Health Check warning notice (#10194 ) When the Consul serf health check is failing, this means that the health checks registered with the agent may no longer be correct. Therefore we show a notice to the user when we detect that the serf health check is failing both for the health check listing for nodes and for service instances. There were a few little things we fixed up whilst we were here: - We use our @replace decorator to replace an empty Type with serf in the model. - We noticed that ServiceTags can be null, so we replace that with an empty array. - We added docs for both our Notice component and the Consul::HealthCheck::List component. Notice now defaults to @type=info.	4 years ago
Kenia	ecbeaa87c1	ui: Add conditionals to Lock Session list items (#10121 ) * Add conditionals to Lock Session list items * Add changelog * Show ID in details if there is a name to go in title * Add copy-button if ID is in the title * Update TTL conditional * Update .changelog/10121.txt Co-authored-by: John Cowen <johncowen@users.noreply.github.com> Co-authored-by: John Cowen <johncowen@users.noreply.github.com>	4 years ago
John Cowen	90d1a307ef	ui: Fix nspace spacing issue (#10157 ) This fixes the spacing bug in nspaces only by only showing Description if the namespace has one, and removing the extra 2 pixel margin of dds for when dts aren't rendered/don't exist.	4 years ago
John Cowen	0e4db3f242	ui: Adds ability to show a 'partial' list in list-collections (#10174 ) * ui: Add support for showing partial lists in ListCollection * Add CSS for partial 'View more' button, and move all CSS to /components * Enable partial view for intention permissions	4 years ago
John Cowen	da78615b58	ui: Remove top border from Consul::AuthMethod::List (#10195 )	4 years ago
John Cowen	e3f9a0f8ee	ui: Loader amends/improvements (#10181 ) * ui: Loader amends/improvements 1. Create a JS compatible template only 'glimmer' component so we can use it with or without glimmer. 2. Add a set of `rose` colors. 3. Animate the brand loader to keep it centered when the side navigation appears. 4. Tweak the color of Consul::Loader to use a 'rose' color. 5. Move everything loader related to the `app/components/` folder and add docs.	4 years ago
John Cowen	5400c81e29	ui: [BUGFIX] De-duplicate Tag rendering (#10186 ) * Add some tests for duplicated and non-duplicated tags * Ensure tags get de-duped and add docs * Update docs to include info on the recursive-ness	4 years ago
Mike Morris	9801551f7c	ui: add data-enterprise-logo to logo svg for enterprise	4 years ago

1 2 3 4 5 ...

637 Commits (add-patches)