Conversation
Related to #175 Draft of DDL monitoring to prevent issues such as users being able to drop their own schema. Committing to get early feedback.
Changed order of ALTER SYSTEM statements Fixes #202
Does not yet return inserted numbers
Fixes #175 Prevent student users from executing DROP SCHEMA and DROP OWNED BY
Correct incorrect script file name in a comment
Simplifies code in isStudent, isInstructor, and isDBManager functions within addClassDBRolesMgmtCore.sql Fixes #208
The original script was attempting to do too much, and was unnecessarily marked "core"
Implement a ddl-start event trigger and associated functions to allow/disallow student-initiated schema drop
Tests functionality in the script addDisallowSchemaDropReco.sql
Changed the CreateXYZ functions to use the default password policy and also raise warning if Intial password was passed
Test Script were changed to support the new CreateXYZ Functions
This reverts commit 3bc1b58.
This reverts commit c9a5259.
Added a more detailed warning and hint. Also changed the call to createRole to not use $1 for the password.
Reorganized IF statement
Fix testUserMgmt.sql
Involved fixing typos, updating API, and removing type casting to prevent a NOTICE from being raised.
Changed comments to refer to updated filenames
Added client_min_messages to addFrequentViewsReco
This reverts commit 5b43f25. Accidentally pushed to dev...
Added client_min_messages to addFrequentViewsReco
update file names with correct names
Looks like CURRENT_DATE was necessary after all
Added getSessionID() to uninstaller
Update and improve privilege tests (M3)
Fix testConnectionActivityLogging.psql when importing old logs
Instructors, DBManagers, and Students can now create schemas. Students still cannot drop schemas. Privilege tests were updated to test this functionality.
Allow creation of schemas by ClassDB users
This reverts commit 00e8acd.
Added m3 docs
Member
|
233 commits since the last release. Nicely done team. |
KevinKelly25
approved these changes
Jun 23, 2018
Contributor
KevinKelly25
left a comment
There was a problem hiding this comment.
Good to Merge. Congrats guys!
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
4 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.
This PR merges all changes included in m3 into master.