-
Notifications
You must be signed in to change notification settings - Fork 997
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
[sp_Blitz] [Bug] Permissions checks are not working as intended #3377
Comments
…dd some messages to the user
…it_3377-xp_readreg #3377 Moved the xp_regread calls back to the check and add some messages to the user
Looks good! Thanks for the pull request. Merging into the dev branch, will be in the next release with credit to you in the release notes. |
…it_3377-msdb_Permissions #3377 Expanded msdb permission checks
…it_3377-sp_MSgetalertinfo #3377 Added a check for sp_MSgetalertinfo
Thanks for the modified pull request! Looks good, works on my machine, heh. Merging into dev, will be in the next release with credit to you in the release notes. If I don't remember "Martin", remind me again the next time you see me live, heh. I'm terrible with names, and Montro1981 is so memorable, oddly! |
Version of the script
8.17
20231010
What is the current behavior?
Mea culpa, some of the fixes I made for the permission checks are not working.
😢
What is the expected behavior?
Fix the code to correct the issue I introduced.
@powerScheme
alright after the first error. The cpu speed not so well. Going to ISNULL the speed to UNKWWill be separate Pull requests per fix.
Which versions of SQL Server and which OS are affected by this issue? Did this work in previous versions of our procedures?
Same errors different day, thus more fixes.
The text was updated successfully, but these errors were encountered: