Commit Graph

2 Commits

Author SHA1 Message Date
Nancy 456f7133c1
check mgmt store and norman store before throwing resource not found (#9380)
* check mgmt store and norman store before throwing resource not found

* Final attempt at fixing resource check in authentication middleware
- Some resource pages don't use the product's store. This happens when..
  - the product changes the store type of a resource via product config `typeStoreMap`
    - explorer project, cluster binding and project binding pages
  - the resource detail / list overrides the store directly with store-override
    - create api key (this isn't in the product / type / id world though)
    - cloud credential create / edit / view
- To fix this
  - ensure we use the correct getter to fetch the store a resource might be in
    - covers typeStoreMap case
  - avoid using `store-override` param for resouces in product/ type / id world
    - covers cloud cred world (use correct typeStoreMap) instead
    - also maake sure we use resourceoverride to get the correct store for a resource
- I'm trying to make sure we support the generic case to avoid breaking extensions which would use the generic inStore toolset
- If this fix doesn't work, we should remove all checks for resources from authentication and instead return to checking for the resource type in resource list & detail components


Tested user with project role in downstream cluster
- cluster instance create / view
- cluster project list & view (saves will fail due to permissions, but screens are ok)
- cluster membership tab (and add page), project membership tab
- cloud creds list, create (and cancel), edit
- places that use resource-override
  - create api key, auth config, monitoring resources
- refresh on all of above

---------

Co-authored-by: Richard Cox <richard.cox@suse.com>
2023-07-20 13:29:55 -07:00
Richard Cox 25acc1f8eb Plugins: Rejig files to new shell locations (#5623)
* Doc

* v0.7.0

* v0.7.1

* v0.7.2

* Add creators

* v0.1.14

* v0.1.17

* Add Yarn link

* Tidy ups and typos

* Allow models to be loaded from plugins

* v0.1.15

* v0.1.18

* v0.6.6

* Improve readme, fix cyperss log msg

* v0.1.17

* v0.1.19

* v0.6.7

* v0.6.8

* Fix logo ref when in shell

* Fix lint issues

* Fix error in example

* Fix script to work on linux

* Add ability to revert moves without losing changes

* Fix bug with custom models in a plugin

* Fix build of UI packages

* Add dist-pkg to .eslintignore

* Pull out util from extend-router to prevent router being pulled in UI packages

* Update PLUGINS.md

* Remove duplicate dependency

* Reduce size of built UI packages

* Share codemirror. Other tidy ups

* Further improvements

* Tidy ups to support i18n in plugins

* Clen up add comments

* More clean-ups and comments added

* Rename from extension to plugin

* Missed file in rename

* v0.6.9

* TIdy ups following rename

* v0.1.20

* v0.1.21

* More refactor and tidy up

* v0.1.22

* v0.1.18

* v0.1.19

* v0.1.23

* v0.6.10

* Version Packages. Improve naming. Unload.

* v0.6.11

* v0.1.20

* v0.1.21

* v0.1.24

* v0.6.12

* v0.1.25

* v0.1.22

* v0.6.13

* Fix issues when plugin is builtin

* Add missing files

* Fix lint issues and watcher ignores

* Fixes following review

* v0.1.28

* v0.1.31

* v0.6.20

* Fix coer.js version

* Fix bug where plugins included via npm don't work

* Changes post merge

* Move plugins doc to dev guide, add note at top of README

* Update cypress version

* Add note about the reset --hard in rejig -d

* Fixes post merge

* Rename @ranch to @rancher and ad plugins

* Improve routes support, add package assets support

* Add uninstall hooks and pass interal opts

* Fix rejig script

* Minor fixes

* Routing tweaks
- attempted to improve `addRoutes` typing, failed a lot
- improved typings in plugins.ts
- added "@pkg/*" entry and vue-shim (for importing components) to the pkg creator
- fixed some linting
- safely fail when Verdaccio isn't running
- fixed some typos

* Remove frontmatter-markdown-loader reference from nuxt-config

* Revert "Remove frontmatter-markdown-loader reference from nuxt-config"

This reverts commit 29ef6f2f7b.

* Exclude creators from dashboard build

* Revert "Exclude creators from dashboard build"

This reverts commit 8ede93ee7c.

* Ingore creators via tsconfig
- We're still susceptible with vue shims in multiple packages
- Need to determine why packages can't use shim from route
- Probably need to move `./vue-shim.d.ts` in to `./shell`. Needs testing with a pkg that imports a component

* After rejig

Co-authored-by: Neil MacDougall <nmacdougall@suse.com>
Co-authored-by: Neil MacDougall <nwmac@users.noreply.github.com>
2022-05-10 10:04:02 +01:00