it-swarm.com.ru

Используйте тип Enum в качестве параметра значения для @ RolesAllowed-Annotation

Я разрабатываю корпоративное приложение Java, в настоящее время выполняю Java EE средства безопасности, чтобы ограничить доступ к определенным функциям для определенных пользователей. Я настроил сервер приложений и все, и теперь я использую аннотацию RolesAllowed для защиты методов:

@Documented
@Retention (RUNTIME)
@Target({TYPE, METHOD})
public @interface RolesAllowed {
    String[] value();
}

Когда я использую аннотацию, как это, она работает нормально:

@RolesAllowed("STUDENT")
public void update(User p) { ... }

Но это не то, чего я хочу, так как здесь нужно использовать строку, рефакторинг становится трудным, и могут произойти опечатки. Поэтому вместо использования строки я бы хотел использовать значение Enum в качестве параметра для этой аннотации. Enum выглядит так:

public enum RoleType {
    STUDENT("STUDENT"),
    TEACHER("TEACHER"),
    DEANERY("DEANERY");

    private final String label;

    private RoleType(String label) {
        this.label = label;
    }

    public String toString() {
        return this.label;
    }
}

Поэтому я попытался использовать Enum в качестве параметра следующим образом:

@RolesAllowed(RoleType.DEANERY.name())
public void update(User p) { ... }

Но затем я получаю следующую ошибку компилятора, хотя Enum.name просто возвращает строку (которая всегда постоянна, не так ли?).

Значение атрибута аннотации RolesAllowed.value должно быть константным выражением`

Следующее, что я попробовал, было добавить дополнительную заключительную строку в мой Enum:

public enum RoleType {
    ...
    public static final String STUDENT_ROLE = STUDENT.toString();
    ...
}

Но это также не работает как параметр, что приводит к той же ошибке компилятора:

// The value for annotation attribute RolesAllowed.value must be a constant expression
@RolesAllowed(RoleType.STUDENT_ROLE)

Как мне добиться желаемого поведения? Я даже реализовал свой собственный перехватчик, чтобы использовать свои собственные аннотации, что прекрасно, но слишком много строк кода для такой маленькой проблемы.

ОТКАЗ ОТ ОТВЕТСТВЕННОСТИ

Этот вопрос изначально был Scala вопрос. Я обнаружил, что Scala не является источником проблемы, поэтому сначала я попытаюсь сделать это на Java.

53
ifischer

Я не думаю, что ваш подход к использованию enums сработает. Я обнаружил, что ошибка компилятора исчезла, если я изменил поле STUDENT_ROLE в вашем последнем примере на постоянную строку, а не на выражение:

public enum RoleType { 
  ...
  public static final String STUDENT_ROLE = "STUDENT";
  ...
}

Тем не менее, это означает, что значения перечисления нигде не будут использоваться, потому что вместо этого вы будете использовать строковые константы в аннотациях.

Мне кажется, что было бы лучше, если бы ваш класс RoleType не содержал ничего, кроме связки статических конечных констант String.


Чтобы понять, почему ваш код не компилируется, я заглянул в Спецификация языка Java (JLS). В JLS для аннотации говорится, что для аннотации с параметром типа T и значением V,

если T является примитивным типом или String, V это постоянное выражение.

A константное выражение включает, помимо прочего,

Полные имена формы TypeName . Идентификатор , который ссылается на постоянные переменные

а постоянная переменная определяется как

переменная примитивного типа или типа String, которая является окончательной и инициализируется константным выражением во время компиляции

31
Luke Woodward

Как насчет этого?

public enum RoleType {
    STUDENT(Names.STUDENT),
    TEACHER(Names.TEACHER),
    DEANERY(Names.DEANERY);

    public class Names{
        public static final String STUDENT = "Student";
        public static final String TEACHER = "Teacher";
        public static final String DEANERY = "Deanery";
    }

    private final String label;

    private RoleType(String label) {
        this.label = label;
    }

    public String toString() {
        return this.label;
    }
}

И в аннотации вы можете использовать его как

@RolesAllowed(RoleType.Names.DEANERY)
public void update(User p) { ... }

Одна небольшая проблема заключается в том, что для любой модификации мы должны измениться в двух местах. Но так как они находятся в одном файле, его вряд ли можно будет пропустить. В свою очередь, мы получаем преимущество от того, что не используем необработанные строки и избегаем сложного механизма.

Или это звучит совершенно глупо? :)

18
Samiron

Вот решение с использованием дополнительного интерфейса и метааннотации. Я включил вспомогательный класс, чтобы помочь сделать рефлексию, чтобы получить типы ролей из набора аннотаций, и небольшой тест для этого:

/**
 * empty interface which must be implemented by enums participating in
 * annotations of "type" @RolesAllowed.
 */
public interface RoleType {
    public String toString();
}

/** meta annotation to be applied to annotations that have enum values implementing RoleType. 
 *  the value() method should return an array of objects assignable to RoleType*.
 */
@Retention(RetentionPolicy.RUNTIME)
@Target({ANNOTATION_TYPE})
public @interface RolesAllowed { 
    /* deliberately empty */ 
}

@RolesAllowed
@Retention(RetentionPolicy.RUNTIME)
@Target({TYPE, METHOD})
public @interface AcademicRolesAllowed {
    public AcademicRoleType[] value();
}

public enum AcademicRoleType implements RoleType {
    STUDENT, TEACHER, DEANERY;
    @Override
    public String toString() {
        return name();
    }
}


public class RolesAllowedUtil {

    /** get the array of allowed RoleTypes for a given class **/
    public static List<RoleType> getRoleTypesAllowedFromAnnotations(
            Annotation[] annotations) {
        List<RoleType> roleTypesAllowed = new ArrayList<RoleType>();
        for (Annotation annotation : annotations) {
            if (annotation.annotationType().isAnnotationPresent(
                    RolesAllowed.class)) {
                RoleType[] roleTypes = getRoleTypesFromAnnotation(annotation);
                if (roleTypes != null)
                    for (RoleType roleType : roleTypes)
                        roleTypesAllowed.add(roleType);
            }
        }
        return roleTypesAllowed;
    }

    public static RoleType[] getRoleTypesFromAnnotation(Annotation annotation) {
        Method[] methods = annotation.annotationType().getMethods();
        for (Method method : methods) {
            String name = method.getName();
            Class<?> returnType = method.getReturnType();
            Class<?> componentType = returnType.getComponentType();
            if (name.equals("value") && returnType.isArray()
                    && RoleType.class.isAssignableFrom(componentType)) {
                RoleType[] features;
                try {
                    features = (RoleType[]) (method.invoke(annotation,
                            new Object[] {}));
                } catch (Exception e) {
                    throw new RuntimeException(
                            "Error executing value() method in "
                                    + annotation.getClass().getCanonicalName(),
                            e);
                }
                return features;
            }
        }
        throw new RuntimeException(
                "No value() method returning a RoleType[] type "
                        + "was found in annotation "
                        + annotation.getClass().getCanonicalName());
    }

}

public class RoleTypeTest {

    @AcademicRolesAllowed({DEANERY})
    public class DeaneryDemo {

    }

    @Test
    public void testDeanery() {
        List<RoleType> roleTypes = RolesAllowedUtil.getRoleTypesAllowedFromAnnotations(DeaneryDemo.class.getAnnotations());
        assertEquals(1, roleTypes.size());
    }
}
9
anomolos

Я решил эту проблему, добавив аннотацию @RoleTypesAllowed и добавив источник метаданных. Это работает очень хорошо, если существует только один тип enum, который необходимо поддерживать. Для нескольких типов перечисления см. Пост Anomolos.

В приведенном ниже RoleType моя роль enum.

@Documented
@Target({ElementType.TYPE, ElementType.METHOD})
@Retention(RetentionPolicy.RUNTIME)
public @interface RoleTypesAllowed {
  RoleType[] value();
}

Затем я добавил следующий источник метаданных в Spring ...

@Slf4j
public class CemsRolesAllowedMethodSecurityMetadataSource
    extends AbstractFallbackMethodSecurityMetadataSource {

  protected Collection<ConfigAttribute> findAttributes(Class<?> clazz) {
    return this.processAnnotations(clazz.getAnnotations());
  }

  protected Collection<ConfigAttribute> findAttributes(Method method, Class<?> targetClass) {
    return this.processAnnotations(AnnotationUtils.getAnnotations(method));
  }

  public Collection<ConfigAttribute> getAllConfigAttributes() {
    return null;
  }

  private List<ConfigAttribute> processAnnotations(Annotation[] annotations) {
    if (annotations != null && annotations.length != 0) {
      List<ConfigAttribute> attributes = new ArrayList();

      for (Annotation a : annotations) {
        if (a instanceof RoleTypesAllowed) {
          RoleTypesAllowed ra = (RoleTypesAllowed) a;
          RoleType[] alloweds = ra.value();
          for (RoleType allowed : alloweds) {
            String defaultedAllowed = new RoleTypeGrantedAuthority(allowed).getAuthority();
            log.trace("Added role attribute: {}", defaultedAllowed);
            attributes.add(new SecurityConfig(defaultedAllowed));
          }
          return attributes;
        }
      }
    }
    return null;
  }
}
0
John B