From 7c94417d2fbd4d7455965a6785d889882b5ea429 Mon Sep 17 00:00:00 2001 From: Martin Desharnais Date: Sun, 13 Feb 2022 18:26:35 +0100 Subject: [PATCH] Return null on illegal field acces issue #940 --- .../jknack/handlebars/context/FieldValueResolver.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/handlebars/src/main/java/com/github/jknack/handlebars/context/FieldValueResolver.java b/handlebars/src/main/java/com/github/jknack/handlebars/context/FieldValueResolver.java index 1b7786727..e1e892221 100644 --- a/handlebars/src/main/java/com/github/jknack/handlebars/context/FieldValueResolver.java +++ b/handlebars/src/main/java/com/github/jknack/handlebars/context/FieldValueResolver.java @@ -212,10 +212,13 @@ public boolean matches(final FieldWrapper field, final String name) { protected Object invokeMember(final FieldWrapper field, final Object context) { try { return field.get(context); + } catch (IllegalAccessException ex) { + // Illegal access to field is expected for classes from external modules + return null; } catch (Exception ex) { throw new IllegalStateException( - "Shouldn't be illegal to access field '" + field.getName() - + "'", ex); + "Shouldn't be illegal to access field '" + field.getName() + + "'", ex); } }