Add a 'security' keyword to all security guides

[nderwin]

Add a security keyword to all security guides, and remove listing for the JWT guide from the web category

(related to quarkusio/quarkus#20622)

Signed-off-by: Nathan Erwin [email protected]

[github-actions]

♻️ PR Preview f627d0b has been successfully destroyed since this PR has been closed.

_{🤖 By surge-preview}

[sberyozkin]

Hi @nderwin thanks for this PR - it does look better, see the PR preview link but it does not show all the guides available under Security category when I search for security, can you please add a security keyword to all the guides in the security category, I think the keywords should support more than one word, either as comma separated values or an array

[nderwin]

@sberyozkin Sure, I'll add the keyword to the others, I just wasn't sure how many to add it to originally.

Should the other guide YAML files have these changes applied, or should it only be applied to the guides-latest.yaml file?

[sberyozkin]

@nderwin

Sure, I'll add the keyword to the others, I just wasn't sure how many to add it to originally.

IMHO all the guides we see under Security category should be shown when a user who'd like to find about Quarkus Security types security, that way it will be consistent, same guides are shown whichever way one starts looking for them...

Should the other guide YAML files have these changes applied, or should it only be applied to the guides-latest.yaml file?

Not sure, Guillaume, @gsmet, can you clarify please when you get a chance ?

[sberyozkin]

@nderwin please squash the commits as well

[sberyozkin]

@nderwin I've looked at 8b0fe6d, I think the same should be done with this PR, can you please also update quarkus-main ?
Also, please update the title and commit message, may be Add a 'security' keyword to all security guides ?

[sberyozkin]

@nderwin thanks, looks good to me, the last minor problem the commit message, it reads removed listing for "Using JWT RBAC" from the web category which is part of this PR but it is primarily about the security keyword, it is probably OK to leave it as it is as the title reflects it, but if you don't mind then please update as well to Add a 'security' keyword to all security guides and remove listing for the JWT guide from the web category or something like that.

I'd like to merge tomorrow in any case, @gsmet if you have any concerns let me know please, IMHO this update is needed.

[sberyozkin]

Waiting till tomorrow for Guillaume to have a look

[nderwin]

@sberyozkin I updated the commit message and the PR description; how's it look now?

If accepted and merged, could I persuade you to put a hacktoberfest-accepted label on the PR?

[sberyozkin]

@nderwin Sure, thanks for your help