{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":368627981,"defaultBranch":"master","name":"server","ownerLogin":"vgrassia","currentUserCanPush":false,"isFork":true,"isEmpty":false,"createdAt":"2021-05-18T18:18:57.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/593223?v=4","public":true,"private":false,"isOrgOwned":false},"refInfo":{"name":"","listCacheKey":"v0:1702323441.0","currentOid":""},"activityList":{"items":[{"before":"5152e4bc2c800757df8cbd492cd3b2b2f1de39e6","after":"5a6a6ce4ce166bace59fe9bbd41394c16dec19d0","ref":"refs/heads/master","pushedAt":"2023-12-12T13:49:45.000Z","pushType":"push","commitsCount":3,"pusher":{"login":"pull[bot]","name":null,"path":"/apps/pull","primaryAvatarUrl":"https://avatars.githubusercontent.com/in/12910?s=80&v=4"},"commit":{"message":"Add commitMessagePrefix to package.json (#3529)","shortMessageHtmlLink":"Add commitMessagePrefix to package.json (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2030461488\" data-permission-text=\"Title is private\" data-url=\"https://github.com/bitwarden/server/issues/3529\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/bitwarden/server/pull/3529/hovercard\" href=\"https://github.com/bitwarden/server/pull/3529\">bitwarden#3529</a>)"}},{"before":"e6ce9ff0ce4f6f7197b8e815bf8009ee50ccab11","after":"5152e4bc2c800757df8cbd492cd3b2b2f1de39e6","ref":"refs/heads/master","pushedAt":"2023-12-12T06:03:46.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"pull[bot]","name":null,"path":"/apps/pull","primaryAvatarUrl":"https://avatars.githubusercontent.com/in/12910?s=80&v=4"},"commit":{"message":"[deps] Vault: Update AngleSharp to v1.0.7 (#3539)\n\nCo-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>","shortMessageHtmlLink":"[deps] Vault: Update AngleSharp to v1.0.7 (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2033504165\" data-permission-text=\"Title is private\" data-url=\"https://github.com/bitwarden/server/issues/3539\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/bitwarden/server/pull/3539/hovercard\" href=\"https://github.com/bitwarden/server/pull/3539\">bitwarden#3539</a>)"}},{"before":"9eefecf54a33b13137121a89ae85f07ae55a1a2e","after":null,"ref":"refs/heads/test","pushedAt":"2023-12-11T19:37:21.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"vgrassia","name":"Vince Grassia","path":"/vgrassia","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/593223?s=80&v=4"}},{"before":null,"after":"9b99b89f8020f3d1952d7ab42cef15548c8d9371","ref":"refs/heads/test-workflow","pushedAt":"2023-12-11T19:30:39.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"vgrassia","name":"Vince Grassia","path":"/vgrassia","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/593223?s=80&v=4"},"commit":{"message":"Test version bump","shortMessageHtmlLink":"Test version bump"}},{"before":"dba7a79ad248ed6ebb494cffd570b4ea3a6c1b3d","after":"e6ce9ff0ce4f6f7197b8e815bf8009ee50ccab11","ref":"refs/heads/master","pushedAt":"2023-12-11T19:27:32.000Z","pushType":"push","commitsCount":60,"pusher":{"login":"vgrassia","name":"Vince Grassia","path":"/vgrassia","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/593223?s=80&v=4"},"commit":{"message":"[AC-1721] Include limit collection creation/deletion in license file (#3388)\n\n* [AC-1117] Add manage permission (#3126)\r\n\r\n* Update sql files to add Manage permission\r\n\r\n* Add migration script\r\n\r\n* Rename collection manage migration file to remove duplicate migration date\r\n\r\n* Migrations\r\n\r\n* Add manage to models\r\n\r\n* Add manage to repository\r\n\r\n* Add constraint to Manage columns\r\n\r\n* Migration lint fixes\r\n\r\n* Add manage to OrganizationUserUserDetails_ReadWithCollectionsById\r\n\r\n* Add missing manage fields\r\n\r\n* Add 'Manage' to UserCollectionDetails\r\n\r\n* Use CREATE OR ALTER where possible\r\n\r\n* [AC-1374] Limit collection creation/deletion to Owner/Admin (#3145)\r\n\r\n* feat: update org table with new column, write migration, refs AC-1374\r\n\r\n* feat: update views with new column, refs AC-1374\r\n\r\n* feat: Alter sprocs (org create/update) to include new column, refs AC-1374\r\n\r\n* feat: update entity/data/request/response models to handle new column, refs AC-1374\r\n\r\n* feat: update necessary Provider related views during migration, refs AC-1374\r\n\r\n* fix: update org create to default new column to false, refs AC-1374\r\n\r\n* feat: added new API/request model for collection management and removed property from update request model, refs AC-1374\r\n\r\n* fix: renamed migration script to be after secrets manage beta column changes, refs AC-1374\r\n\r\n* fix: dotnet format, refs AC-1374\r\n\r\n* feat: add ef migrations to reflect mssql changes, refs AC-1374\r\n\r\n* fix: dotnet format, refs AC-1374\r\n\r\n* feat: update API signature to accept Guid and explain Cd verbiage, refs AC-1374\r\n\r\n* fix: merge conflict resolution\r\n\r\n* [AC-1174] CollectionUser and CollectionGroup authorization handlers (#3194)\r\n\r\n* [AC-1174] Introduce BulkAuthorizationHandler.cs\r\n\r\n* [AC-1174] Introduce CollectionUserAuthorizationHandler\r\n\r\n* [AC-1174] Add CreateForNewCollection CollectionUser requirement\r\n\r\n* [AC-1174] Add some more details to CollectionCustomization\r\n\r\n* [AC-1174] Formatting\r\n\r\n* [AC-1174] Add CollectionGroupOperation.cs\r\n\r\n* [AC-1174] Introduce CollectionGroupAuthorizationHandler.cs\r\n\r\n* [AC-1174] Cleanup CollectionFixture customization\r\n\r\nImplement and use re-usable extension method to support seeded Guids\r\n\r\n* [AC-1174] Introduce WithValueFromList AutoFixtureExtensions\r\n\r\nModify CollectionCustomization to use multiple organization Ids for auto generated test data\r\n\r\n* [AC-1174] Simplify CollectionUserAuthorizationHandler.cs\r\n\r\nModify the authorization handler to only perform authorization logic. Validation logic will need to be handled by any calling commands/controllers instead.\r\n\r\n* [AC-1174] Introduce shared CollectionAccessAuthorizationHandlerBase\r\n\r\nA shared base authorization handler was created for both CollectionUser and CollectionGroup resources, as they share the same underlying management authorization logic.\r\n\r\n* [AC-1174] Update CollectionUserAuthorizationHandler and CollectionGroupAuthorizationHandler to use the new CollectionAccessAuthorizationHandlerBase class\r\n\r\n* [AC-1174] Formatting\r\n\r\n* [AC-1174] Cleanup typo and redundant ToList() call\r\n\r\n* [AC-1174] Add check for provider users\r\n\r\n* [AC-1174] Reduce nested loops\r\n\r\n* [AC-1174] Introduce ICollectionAccess.cs\r\n\r\n* [AC-1174] Remove individual CollectionGroup and CollectionUser auth handlers and use base class instead\r\n\r\n* [AC-1174] Tweak unit test to fail minimally\r\n\r\n* [AC-1174] Reorganize authorization handlers in Core project\r\n\r\n* [AC-1174] Introduce new AddCoreAuthorizationHandlers() extension method\r\n\r\n* [AC-1174] Move CollectionAccessAuthorizationHandler into Api project\r\n\r\n* [AC-1174] Move CollectionFixture to Vault folder\r\n\r\n* [AC-1174] Rename operation to CreateUpdateDelete\r\n\r\n* [AC-1174] Require single organization for collection access authorization handler\r\n\r\n- Add requirement that all target collections must belong to the same organization\r\n- Simplify logic related to multiple organizations\r\n- Update tests and helpers\r\n- Use ToHashSet to improve lookup time\r\n\r\n* [AC-1174] Fix null reference exception\r\n\r\n* [AC-1174] Throw bad request exception when collections belong to different organizations\r\n\r\n* [AC-1174] Switch to CollectionAuthorizationHandler instead of CollectionAccessAuthorizationHandler to reduce complexity\r\n\r\n* Fix improper merge conflict resolution\r\n\r\n* fix: add permission check for collection management api, refs AC-1647 (#3252)\r\n\r\n* [AC-1125] Enforce org setting for creating/deleting collections (#3241)\r\n\r\n* [AC-1117] Add manage permission (#3126)\r\n\r\n* Update sql files to add Manage permission\r\n\r\n* Add migration script\r\n\r\n* Rename collection manage migration file to remove duplicate migration date\r\n\r\n* Migrations\r\n\r\n* Add manage to models\r\n\r\n* Add manage to repository\r\n\r\n* Add constraint to Manage columns\r\n\r\n* Migration lint fixes\r\n\r\n* Add manage to OrganizationUserUserDetails_ReadWithCollectionsById\r\n\r\n* Add missing manage fields\r\n\r\n* Add 'Manage' to UserCollectionDetails\r\n\r\n* Use CREATE OR ALTER where possible\r\n\r\n* [AC-1374] Limit collection creation/deletion to Owner/Admin (#3145)\r\n\r\n* feat: update org table with new column, write migration, refs AC-1374\r\n\r\n* feat: update views with new column, refs AC-1374\r\n\r\n* feat: Alter sprocs (org create/update) to include new column, refs AC-1374\r\n\r\n* feat: update entity/data/request/response models to handle new column, refs AC-1374\r\n\r\n* feat: update necessary Provider related views during migration, refs AC-1374\r\n\r\n* fix: update org create to default new column to false, refs AC-1374\r\n\r\n* feat: added new API/request model for collection management and removed property from update request model, refs AC-1374\r\n\r\n* fix: renamed migration script to be after secrets manage beta column changes, refs AC-1374\r\n\r\n* fix: dotnet format, refs AC-1374\r\n\r\n* feat: add ef migrations to reflect mssql changes, refs AC-1374\r\n\r\n* fix: dotnet format, refs AC-1374\r\n\r\n* feat: update API signature to accept Guid and explain Cd verbiage, refs AC-1374\r\n\r\n* feat: created collection auth handler/operations, added LimitCollectionCdOwnerAdmin to CurrentContentOrganization, refs AC-1125\r\n\r\n* feat: create vault service collection extensions and register with base services, refs AC-1125\r\n\r\n* feat: deprecated CurrentContext.CreateNewCollections, refs AC-1125\r\n\r\n* feat: deprecate DeleteAnyCollection for single resource usages, refs AC-1125\r\n\r\n* feat: move service registration to api, update references, refs AC-1125\r\n\r\n* feat: add bulk delete authorization handler, refs AC-1125\r\n\r\n* feat: always assign user and give manage access on create, refs AC-1125\r\n\r\n* fix: updated CurrentContextOrganization type, refs AC-1125\r\n\r\n* feat: combined existing collection authorization handlers/operations, refs AC-1125\r\n\r\n* fix: OrganizationServiceTests -> CurrentContentOrganization typo, refs AC-1125\r\n\r\n* fix: format, refs AC-1125\r\n\r\n* fix: update collection controller tests, refs AC-1125\r\n\r\n* fix: dotnet format, refs AC-1125\r\n\r\n* feat: removed extra BulkAuthorizationHandler, refs AC-1125\r\n\r\n* fix: dotnet format, refs AC-1125\r\n\r\n* fix: change string to guid for org id, update bulk delete request model, refs AC-1125\r\n\r\n* fix: remove delete many collection check, refs AC-1125\r\n\r\n* fix: clean up collection auth handler, refs AC-1125\r\n\r\n* fix: format fix for CollectionOperations, refs AC-1125\r\n\r\n* fix: removed unnecessary owner check, add org null check to custom permission validation, refs AC-1125\r\n\r\n* fix: remove unused methods in CurrentContext, refs AC-1125\r\n\r\n* fix: removed obsolete test, fixed failling delete many test, refs AC-1125\r\n\r\n* fix: CollectionAuthorizationHandlerTests fixes, refs AC-1125\r\n\r\n* fix: OrganizationServiceTests fix broken test by mocking GetOrganization, refs AC-1125\r\n\r\n* fix: CollectionAuthorizationHandler - remove unused repository, refs AC-1125\r\n\r\n* feat: moved UserId null check to common method, refs AC-1125\r\n\r\n* fix: updated auth handler tests to remove dependency on requirement for common code checks, refs AC-1125\r\n\r\n* feat: updated conditionals/comments for create/delete methods within colleciton auth handler, refs AC-1125\r\n\r\n* feat: added create/delete collection auth handler success methods, refs AC-1125\r\n\r\n* fix: new up permissions to prevent excessive null checks, refs AC-1125\r\n\r\n* fix: remove old reference to CreateNewCollections, refs AC-1125\r\n\r\n* fix: typo within ViewAssignedCollections method, refs AC-1125\r\n\r\n---------\r\n\r\nCo-authored-by: Robyn MacCallum <robyntmaccallum@gmail.com>\r\n\r\n* refactor: remove organizationId from CollectionBulkDeleteRequestModel, refs AC-1649 (#3282)\r\n\r\n* [AC-1174] Bulk Collection Management (#3229)\r\n\r\n* [AC-1174] Update SelectionReadOnlyRequestModel to use Guid for Id property\r\n\r\n* [AC-1174] Introduce initial bulk-access collection endpoint\r\n\r\n* [AC-1174] Introduce BulkAddCollectionAccessCommand and validation logic/tests\r\n\r\n* [AC-1174] Add CreateOrUpdateAccessMany method to CollectionRepository\r\n\r\n* [AC-1174] Add event logs for bulk add collection access command\r\n\r\n* [AC-1174] Add User_BumpAccountRevisionDateByCollectionIds and database migration script\r\n\r\n* [AC-1174] Implement EF repository method\r\n\r\n* [AC-1174] Improve null checks\r\n\r\n* [AC-1174] Remove unnecessary BulkCollectionAccessRequestModel helpers\r\n\r\n* [AC-1174] Add unit tests for new controller endpoint\r\n\r\n* [AC-1174] Fix formatting\r\n\r\n* [AC-1174] Remove comment\r\n\r\n* [AC-1174] Remove redundant organizationId parameter\r\n\r\n* [AC-1174] Ensure user and group Ids are distinct\r\n\r\n* [AC-1174] Cleanup tests based on PR feedback\r\n\r\n* [AC-1174] Formatting\r\n\r\n* [AC-1174] Update CollectionGroup alias in the sproc\r\n\r\n* [AC-1174] Add some additional comments to SQL sproc\r\n\r\n* [AC-1174] Add comment explaining additional SaveChangesAsync call\r\n\r\n---------\r\n\r\nCo-authored-by: Thomas Rittson <trittson@bitwarden.com>\r\n\r\n* [AC-1646] Rename LimitCollectionCdOwnerAdmin column (#3300)\r\n\r\n* Rename LimitCollectionCdOwnerAdmin -> LimitCollectionCreationDeletion\r\n\r\n* Rename and bump migration script\r\n\r\n* [AC-1666] Removed EditAnyCollection from Create/Delete permission checks (#3301)\r\n\r\n* fix: remove EditAnyCollection from Create/Delete permission check, refs AC-1666\r\n\r\n* fix: updated comment, refs AC-1666\r\n\r\n* [AC-1669] Bug - Remove obsolete assignUserId from CollectionService.SaveAsync(...) (#3312)\r\n\r\n* fix: remove AssignUserId from CollectionService.SaveAsync, refs AC-1669\r\n\r\n* fix: add manage access conditional before creating collection, refs AC-1669\r\n\r\n* fix: move access logic for create/update, fix all tests, refs AC-1669\r\n\r\n* fix: add CollectionAccessSelection fixture, update tests, update bad reqeuest message, refs AC-1669\r\n\r\n* fix: format, refs AC-1669\r\n\r\n* fix: update null params with specific arg.is null checks, refs Ac-1669\r\n\r\n* fix: update attribute class name, refs AC-1669\r\n\r\n* [AC-1713] [Flexible collections] Add feature flags to server (#3334)\r\n\r\n* Add feature flags for FlexibleCollections and BulkCollectionAccess\r\n\r\n* Flag new routes and behaviour\r\n\r\n---------\r\n\r\nCo-authored-by: Rui Tomé <108268980+r-tome@users.noreply.github.com>\r\n\r\n* Add joint codeownership for auth handlers (#3346)\r\n\r\n* [AC-1717] Update default values for LimitCollectionCreationDeletion (#3365)\r\n\r\n* Change default value in organization create sproc to 1\r\n\r\n* Drop old column name still present in some QA instances\r\n\r\n* Set LimitCollectionCreationDeletion value in code based on feature flag\r\n\r\n* Fix: add missing namespace after merging in master\r\n\r\n* Fix: add missing namespace after merging in master\r\n\r\n* [AC-1683] Fix DB migrations for new Manage permission (#3307)\r\n\r\n* [AC-1683] Update migration script and introduce V2 procedures and types\r\n\r\n* [AC-1683] Update repository calls to use new V2 procedures / types\r\n\r\n* [AC-1684] Update bulk add collection migration script to use new V2 type\r\n\r\n* [AC-1683] Undo Manage changes to more original procedures\r\n\r\n* [AC-1683] Restore whitespace changes\r\n\r\n* [AC-1683] Clarify comments regarding explicit column lists\r\n\r\n* [AC-1683] Update migration script dates\r\n\r\n* [AC-1683] Split the migration script for readability\r\n\r\n* [AC-1683] Re-name SelectReadOnlyArray_V2 to CollectionAccessSelectionType\r\n\r\n* [AC-1648] [Flexible Collections] Bump migration scripts before feature branch merge (#3371)\r\n\r\n* Bump dates on sql migration scripts\r\n\r\n* Bump date on ef migrations\r\n\r\n* feat: update OrganizationLicense (add property, update GetDataBytes, update VerifyData), refs AC-1721\r\n\r\n* feat: update Organization.UpdateFromLicense and SignUpAsync to use value when permittable, refs AC-1721\r\n\r\n* feat: Add cloud-only access for PutCollectionManagement endpoint, refs AC-172\r\n\r\n* feat: add feature flag to organization entity for updating from license, refs AC-1721\r\n\r\n* feat: updated license fixture with new version (14), refs AC-1721\r\n\r\n* feat: disable validity checks for version 14, refs AC-1721\r\n\r\n* fix: dotnet format, refs AC-1721\r\n\r\n* feat: default org license LimitCollectionCreationDeletion to true, refs AC-1721\r\n\r\n---------\r\n\r\nCo-authored-by: Robyn MacCallum <robyntmaccallum@gmail.com>\r\nCo-authored-by: Shane Melton <smelton@bitwarden.com>\r\nCo-authored-by: Thomas Rittson <trittson@bitwarden.com>\r\nCo-authored-by: Thomas Rittson <31796059+eliykat@users.noreply.github.com>\r\nCo-authored-by: Rui Tomé <108268980+r-tome@users.noreply.github.com>","shortMessageHtmlLink":"[AC-1721] Include limit collection creation/deletion in license file (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"1969184905\" data-permission-text=\"Title is private\" data-url=\"https://github.com/bitwarden/server/issues/3388\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/bitwarden/server/pull/3388/hovercard\" href=\"https://github.com/bitwarden/server/pull/3388\">b…</a>"}},{"before":"636a7646a3499699589e834bee5abf0cfd0f047d","after":"dba7a79ad248ed6ebb494cffd570b4ea3a6c1b3d","ref":"refs/heads/master","pushedAt":"2023-11-22T14:48:54.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"pull[bot]","name":null,"path":"/apps/pull","primaryAvatarUrl":"https://avatars.githubusercontent.com/in/12910?s=80&v=4"},"commit":{"message":"[AC-1696] Fix Provider permissions for Flexible Collections (#3381)\n\n* [AC-1117] Add manage permission (#3126)\r\n\r\n* Update sql files to add Manage permission\r\n\r\n* Add migration script\r\n\r\n* Rename collection manage migration file to remove duplicate migration date\r\n\r\n* Migrations\r\n\r\n* Add manage to models\r\n\r\n* Add manage to repository\r\n\r\n* Add constraint to Manage columns\r\n\r\n* Migration lint fixes\r\n\r\n* Add manage to OrganizationUserUserDetails_ReadWithCollectionsById\r\n\r\n* Add missing manage fields\r\n\r\n* Add 'Manage' to UserCollectionDetails\r\n\r\n* Use CREATE OR ALTER where possible\r\n\r\n* [AC-1374] Limit collection creation/deletion to Owner/Admin (#3145)\r\n\r\n* feat: update org table with new column, write migration, refs AC-1374\r\n\r\n* feat: update views with new column, refs AC-1374\r\n\r\n* feat: Alter sprocs (org create/update) to include new column, refs AC-1374\r\n\r\n* feat: update entity/data/request/response models to handle new column, refs AC-1374\r\n\r\n* feat: update necessary Provider related views during migration, refs AC-1374\r\n\r\n* fix: update org create to default new column to false, refs AC-1374\r\n\r\n* feat: added new API/request model for collection management and removed property from update request model, refs AC-1374\r\n\r\n* fix: renamed migration script to be after secrets manage beta column changes, refs AC-1374\r\n\r\n* fix: dotnet format, refs AC-1374\r\n\r\n* feat: add ef migrations to reflect mssql changes, refs AC-1374\r\n\r\n* fix: dotnet format, refs AC-1374\r\n\r\n* feat: update API signature to accept Guid and explain Cd verbiage, refs AC-1374\r\n\r\n* fix: merge conflict resolution\r\n\r\n* [AC-1174] CollectionUser and CollectionGroup authorization handlers (#3194)\r\n\r\n* [AC-1174] Introduce BulkAuthorizationHandler.cs\r\n\r\n* [AC-1174] Introduce CollectionUserAuthorizationHandler\r\n\r\n* [AC-1174] Add CreateForNewCollection CollectionUser requirement\r\n\r\n* [AC-1174] Add some more details to CollectionCustomization\r\n\r\n* [AC-1174] Formatting\r\n\r\n* [AC-1174] Add CollectionGroupOperation.cs\r\n\r\n* [AC-1174] Introduce CollectionGroupAuthorizationHandler.cs\r\n\r\n* [AC-1174] Cleanup CollectionFixture customization\r\n\r\nImplement and use re-usable extension method to support seeded Guids\r\n\r\n* [AC-1174] Introduce WithValueFromList AutoFixtureExtensions\r\n\r\nModify CollectionCustomization to use multiple organization Ids for auto generated test data\r\n\r\n* [AC-1174] Simplify CollectionUserAuthorizationHandler.cs\r\n\r\nModify the authorization handler to only perform authorization logic. Validation logic will need to be handled by any calling commands/controllers instead.\r\n\r\n* [AC-1174] Introduce shared CollectionAccessAuthorizationHandlerBase\r\n\r\nA shared base authorization handler was created for both CollectionUser and CollectionGroup resources, as they share the same underlying management authorization logic.\r\n\r\n* [AC-1174] Update CollectionUserAuthorizationHandler and CollectionGroupAuthorizationHandler to use the new CollectionAccessAuthorizationHandlerBase class\r\n\r\n* [AC-1174] Formatting\r\n\r\n* [AC-1174] Cleanup typo and redundant ToList() call\r\n\r\n* [AC-1174] Add check for provider users\r\n\r\n* [AC-1174] Reduce nested loops\r\n\r\n* [AC-1174] Introduce ICollectionAccess.cs\r\n\r\n* [AC-1174] Remove individual CollectionGroup and CollectionUser auth handlers and use base class instead\r\n\r\n* [AC-1174] Tweak unit test to fail minimally\r\n\r\n* [AC-1174] Reorganize authorization handlers in Core project\r\n\r\n* [AC-1174] Introduce new AddCoreAuthorizationHandlers() extension method\r\n\r\n* [AC-1174] Move CollectionAccessAuthorizationHandler into Api project\r\n\r\n* [AC-1174] Move CollectionFixture to Vault folder\r\n\r\n* [AC-1174] Rename operation to CreateUpdateDelete\r\n\r\n* [AC-1174] Require single organization for collection access authorization handler\r\n\r\n- Add requirement that all target collections must belong to the same organization\r\n- Simplify logic related to multiple organizations\r\n- Update tests and helpers\r\n- Use ToHashSet to improve lookup time\r\n\r\n* [AC-1174] Fix null reference exception\r\n\r\n* [AC-1174] Throw bad request exception when collections belong to different organizations\r\n\r\n* [AC-1174] Switch to CollectionAuthorizationHandler instead of CollectionAccessAuthorizationHandler to reduce complexity\r\n\r\n* Fix improper merge conflict resolution\r\n\r\n* fix: add permission check for collection management api, refs AC-1647 (#3252)\r\n\r\n* [AC-1125] Enforce org setting for creating/deleting collections (#3241)\r\n\r\n* [AC-1117] Add manage permission (#3126)\r\n\r\n* Update sql files to add Manage permission\r\n\r\n* Add migration script\r\n\r\n* Rename collection manage migration file to remove duplicate migration date\r\n\r\n* Migrations\r\n\r\n* Add manage to models\r\n\r\n* Add manage to repository\r\n\r\n* Add constraint to Manage columns\r\n\r\n* Migration lint fixes\r\n\r\n* Add manage to OrganizationUserUserDetails_ReadWithCollectionsById\r\n\r\n* Add missing manage fields\r\n\r\n* Add 'Manage' to UserCollectionDetails\r\n\r\n* Use CREATE OR ALTER where possible\r\n\r\n* [AC-1374] Limit collection creation/deletion to Owner/Admin (#3145)\r\n\r\n* feat: update org table with new column, write migration, refs AC-1374\r\n\r\n* feat: update views with new column, refs AC-1374\r\n\r\n* feat: Alter sprocs (org create/update) to include new column, refs AC-1374\r\n\r\n* feat: update entity/data/request/response models to handle new column, refs AC-1374\r\n\r\n* feat: update necessary Provider related views during migration, refs AC-1374\r\n\r\n* fix: update org create to default new column to false, refs AC-1374\r\n\r\n* feat: added new API/request model for collection management and removed property from update request model, refs AC-1374\r\n\r\n* fix: renamed migration script to be after secrets manage beta column changes, refs AC-1374\r\n\r\n* fix: dotnet format, refs AC-1374\r\n\r\n* feat: add ef migrations to reflect mssql changes, refs AC-1374\r\n\r\n* fix: dotnet format, refs AC-1374\r\n\r\n* feat: update API signature to accept Guid and explain Cd verbiage, refs AC-1374\r\n\r\n* feat: created collection auth handler/operations, added LimitCollectionCdOwnerAdmin to CurrentContentOrganization, refs AC-1125\r\n\r\n* feat: create vault service collection extensions and register with base services, refs AC-1125\r\n\r\n* feat: deprecated CurrentContext.CreateNewCollections, refs AC-1125\r\n\r\n* feat: deprecate DeleteAnyCollection for single resource usages, refs AC-1125\r\n\r\n* feat: move service registration to api, update references, refs AC-1125\r\n\r\n* feat: add bulk delete authorization handler, refs AC-1125\r\n\r\n* feat: always assign user and give manage access on create, refs AC-1125\r\n\r\n* fix: updated CurrentContextOrganization type, refs AC-1125\r\n\r\n* feat: combined existing collection authorization handlers/operations, refs AC-1125\r\n\r\n* fix: OrganizationServiceTests -> CurrentContentOrganization typo, refs AC-1125\r\n\r\n* fix: format, refs AC-1125\r\n\r\n* fix: update collection controller tests, refs AC-1125\r\n\r\n* fix: dotnet format, refs AC-1125\r\n\r\n* feat: removed extra BulkAuthorizationHandler, refs AC-1125\r\n\r\n* fix: dotnet format, refs AC-1125\r\n\r\n* fix: change string to guid for org id, update bulk delete request model, refs AC-1125\r\n\r\n* fix: remove delete many collection check, refs AC-1125\r\n\r\n* fix: clean up collection auth handler, refs AC-1125\r\n\r\n* fix: format fix for CollectionOperations, refs AC-1125\r\n\r\n* fix: removed unnecessary owner check, add org null check to custom permission validation, refs AC-1125\r\n\r\n* fix: remove unused methods in CurrentContext, refs AC-1125\r\n\r\n* fix: removed obsolete test, fixed failling delete many test, refs AC-1125\r\n\r\n* fix: CollectionAuthorizationHandlerTests fixes, refs AC-1125\r\n\r\n* fix: OrganizationServiceTests fix broken test by mocking GetOrganization, refs AC-1125\r\n\r\n* fix: CollectionAuthorizationHandler - remove unused repository, refs AC-1125\r\n\r\n* feat: moved UserId null check to common method, refs AC-1125\r\n\r\n* fix: updated auth handler tests to remove dependency on requirement for common code checks, refs AC-1125\r\n\r\n* feat: updated conditionals/comments for create/delete methods within colleciton auth handler, refs AC-1125\r\n\r\n* feat: added create/delete collection auth handler success methods, refs AC-1125\r\n\r\n* fix: new up permissions to prevent excessive null checks, refs AC-1125\r\n\r\n* fix: remove old reference to CreateNewCollections, refs AC-1125\r\n\r\n* fix: typo within ViewAssignedCollections method, refs AC-1125\r\n\r\n---------\r\n\r\nCo-authored-by: Robyn MacCallum <robyntmaccallum@gmail.com>\r\n\r\n* refactor: remove organizationId from CollectionBulkDeleteRequestModel, refs AC-1649 (#3282)\r\n\r\n* [AC-1174] Bulk Collection Management (#3229)\r\n\r\n* [AC-1174] Update SelectionReadOnlyRequestModel to use Guid for Id property\r\n\r\n* [AC-1174] Introduce initial bulk-access collection endpoint\r\n\r\n* [AC-1174] Introduce BulkAddCollectionAccessCommand and validation logic/tests\r\n\r\n* [AC-1174] Add CreateOrUpdateAccessMany method to CollectionRepository\r\n\r\n* [AC-1174] Add event logs for bulk add collection access command\r\n\r\n* [AC-1174] Add User_BumpAccountRevisionDateByCollectionIds and database migration script\r\n\r\n* [AC-1174] Implement EF repository method\r\n\r\n* [AC-1174] Improve null checks\r\n\r\n* [AC-1174] Remove unnecessary BulkCollectionAccessRequestModel helpers\r\n\r\n* [AC-1174] Add unit tests for new controller endpoint\r\n\r\n* [AC-1174] Fix formatting\r\n\r\n* [AC-1174] Remove comment\r\n\r\n* [AC-1174] Remove redundant organizationId parameter\r\n\r\n* [AC-1174] Ensure user and group Ids are distinct\r\n\r\n* [AC-1174] Cleanup tests based on PR feedback\r\n\r\n* [AC-1174] Formatting\r\n\r\n* [AC-1174] Update CollectionGroup alias in the sproc\r\n\r\n* [AC-1174] Add some additional comments to SQL sproc\r\n\r\n* [AC-1174] Add comment explaining additional SaveChangesAsync call\r\n\r\n---------\r\n\r\nCo-authored-by: Thomas Rittson <trittson@bitwarden.com>\r\n\r\n* [AC-1646] Rename LimitCollectionCdOwnerAdmin column (#3300)\r\n\r\n* Rename LimitCollectionCdOwnerAdmin -> LimitCollectionCreationDeletion\r\n\r\n* Rename and bump migration script\r\n\r\n* [AC-1666] Removed EditAnyCollection from Create/Delete permission checks (#3301)\r\n\r\n* fix: remove EditAnyCollection from Create/Delete permission check, refs AC-1666\r\n\r\n* fix: updated comment, refs AC-1666\r\n\r\n* [AC-1669] Bug - Remove obsolete assignUserId from CollectionService.SaveAsync(...) (#3312)\r\n\r\n* fix: remove AssignUserId from CollectionService.SaveAsync, refs AC-1669\r\n\r\n* fix: add manage access conditional before creating collection, refs AC-1669\r\n\r\n* fix: move access logic for create/update, fix all tests, refs AC-1669\r\n\r\n* fix: add CollectionAccessSelection fixture, update tests, update bad reqeuest message, refs AC-1669\r\n\r\n* fix: format, refs AC-1669\r\n\r\n* fix: update null params with specific arg.is null checks, refs Ac-1669\r\n\r\n* fix: update attribute class name, refs AC-1669\r\n\r\n* [AC-1713] [Flexible collections] Add feature flags to server (#3334)\r\n\r\n* Add feature flags for FlexibleCollections and BulkCollectionAccess\r\n\r\n* Flag new routes and behaviour\r\n\r\n---------\r\n\r\nCo-authored-by: Rui Tomé <108268980+r-tome@users.noreply.github.com>\r\n\r\n* Add joint codeownership for auth handlers (#3346)\r\n\r\n* [AC-1717] Update default values for LimitCollectionCreationDeletion (#3365)\r\n\r\n* Change default value in organization create sproc to 1\r\n\r\n* Drop old column name still present in some QA instances\r\n\r\n* Set LimitCollectionCreationDeletion value in code based on feature flag\r\n\r\n* Fix: add missing namespace after merging in master\r\n\r\n* Fix: add missing namespace after merging in master\r\n\r\n* [AC-1683] Fix DB migrations for new Manage permission (#3307)\r\n\r\n* [AC-1683] Update migration script and introduce V2 procedures and types\r\n\r\n* [AC-1683] Update repository calls to use new V2 procedures / types\r\n\r\n* [AC-1684] Update bulk add collection migration script to use new V2 type\r\n\r\n* [AC-1683] Undo Manage changes to more original procedures\r\n\r\n* [AC-1683] Restore whitespace changes\r\n\r\n* [AC-1683] Clarify comments regarding explicit column lists\r\n\r\n* [AC-1683] Update migration script dates\r\n\r\n* [AC-1683] Split the migration script for readability\r\n\r\n* [AC-1683] Re-name SelectReadOnlyArray_V2 to CollectionAccessSelectionType\r\n\r\n* [AC-1648] [Flexible Collections] Bump migration scripts before feature branch merge (#3371)\r\n\r\n* Bump dates on sql migration scripts\r\n\r\n* Bump date on ef migrations\r\n\r\n* [AC-1696] Rework collection auth handler to account for provider users\r\n\r\n- Add additional check for providers only if the org membership is null, and if the user is a provider, do not early return\r\n- Modify conditionals to use pattern matching to allow for null organization contexts from provider users\r\n- Save the target organization id as a private field to avoid additional parameter passing because org can now be null\r\n\r\n* [AC-1696] Add unit tests to collection auth handler for provider users\r\n\r\nIncludes helper test method/enum for creating an xUnit theory for each collection operation.\r\n\r\n* [AC-1696] Further refactor private methods and remove provider check from public method\r\n\r\nUpdates logic in private methods to only use context.Succeed() to allow for fallback permission checking for provider access.\r\n\r\n* [AC-1696] Ensure the correct organization id is provided when testing\r\n\r\n* [AC-1696] Refactor provider test to remove additional operation enum\r\n\r\n* [AC-1696] Formatting\r\n\r\n---------\r\n\r\nCo-authored-by: Robyn MacCallum <robyntmaccallum@gmail.com>\r\nCo-authored-by: Vincent Salucci <26154748+vincentsalucci@users.noreply.github.com>\r\nCo-authored-by: Vincent Salucci <vincesalucci21@gmail.com>\r\nCo-authored-by: Thomas Rittson <trittson@bitwarden.com>\r\nCo-authored-by: Thomas Rittson <31796059+eliykat@users.noreply.github.com>\r\nCo-authored-by: Rui Tomé <108268980+r-tome@users.noreply.github.com>","shortMessageHtmlLink":"[AC-1696] Fix Provider permissions for Flexible Collections (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"1964548514\" data-permission-text=\"Title is private\" data-url=\"https://github.com/bitwarden/server/issues/3381\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/bitwarden/server/pull/3381/hovercard\" href=\"https://github.com/bitwarden/server/pull/3381\">bitwarde…</a>"}},{"before":null,"after":"4980c5041d76da23bc3662d81f1f3f5f6d6fdc76","ref":"refs/heads/my-docker-builds","pushedAt":"2023-11-22T00:57:43.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"vgrassia","name":"Vince Grassia","path":"/vgrassia","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/593223?s=80&v=4"},"commit":{"message":"Fix path","shortMessageHtmlLink":"Fix path"}},{"before":"c2fe3e494980aa82f9f5864a18f8d2f8d01d35fc","after":"636a7646a3499699589e834bee5abf0cfd0f047d","ref":"refs/heads/master","pushedAt":"2023-11-22T00:42:48.000Z","pushType":"push","commitsCount":520,"pusher":{"login":"vgrassia","name":"Vince Grassia","path":"/vgrassia","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/593223?s=80&v=4"},"commit":{"message":"Bumped version to 2023.10.3 (#3462)\n\nCo-authored-by: bitwarden-devops-bot <106330231+bitwarden-devops-bot@users.noreply.github.com>","shortMessageHtmlLink":"Bumped version to 2023.10.3 (<a class=\"issue-link js-issue-link\" data-error-text=\"Failed to load title\" data-id=\"2004723532\" data-permission-text=\"Title is private\" data-url=\"https://github.com/bitwarden/server/issues/3462\" data-hovercard-type=\"pull_request\" data-hovercard-url=\"/bitwarden/server/pull/3462/hovercard\" href=\"https://github.com/bitwarden/server/pull/3462\">bitwarden#3462</a>)"}}],"hasNextPage":false,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAADyGjEQgA","startCursor":null,"endCursor":null}},"title":"Activity · vgrassia/server"}