feat: Add support for "project_destroy" and "project_rename" hooks

Resolves #80 (closed)

added 4 commits

• bb0d3857 - fix: Add missing type casting of query params
• c7d285e8 - fix: Fix project path param in namespace
• d3960717 - fix: Replace event name check in init validation
• a654f7e8 - feat: Add deletion tracking process

Compare with previous version

added 1 commit

• a5e11936 - feat: Add project_rename process

Compare with previous version

requested review from @malowe

requested review from @cguindon

marked this merge request as ready

Instead of static strings, we should use an enum with a method to cast a name to the enum. A lot of our logic gets way cleaner that way. An example of this is from the membership portal, in the image formats:

    public enum ImageStoreFormats {
        SMALL, LARGE, PRINT, WEB, WEB_SRC;

        public String getName() {
            return this.name().toLowerCase();
        }

        public static ImageStoreFormat getFormat(String name) {
            if (name == null) {
                return null;
            }
            return Stream.of(values()).filter(v -> name.equalsIgnoreCase(v.getName())).findFirst().orElse(null);
        }
    }

With this sort of thing, if you replace the null return with an UNSUPPORTED enum, you could then do something like this:

SupportedEventType t = getSupportedEventType(theString);
switch(t) {
  case PROJECT_CREATE:
    doTheThing();
    break;
  ...
  case UNSUPPORTED:
    LOGGER.trace("Dropped event {}", theString);
    break;
}

changed this line in version 4 of the diff

marked this merge request as draft

added 2 commits

• 1780baf6 - feat: Add EventType enum
• d381f15a - feat: Restructure validation to use event enum

Compare with previous version

added 1 commit

• bb923132 - fix: fix session closing issue

Compare with previous version

added 1 commit

• 4af3433f - feat: Update spec to include new field

Compare with previous version

added 1 commit

• 6aaa939e - feat: Add testing for delete and renam hooks

Compare with previous version

marked this merge request as ready

@malowe ready for review :)

merged

mentioned in commit e421d3ac