Apache
/
olingo-odata4
mirror of https://github.com/apache/olingo-odata4.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[OLINGO-659] Lexer recognizes invalid charaters

This commit is contained in:
Christia Holzer 2015-09-15 12:57:31 +02:00
parent 9c3ca381e2
commit d84c3275cb
2 changed files with 46 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
lib/server-core/src/main/antlr4/org/apache/olingo/server/core/uri/antlr/UriLexer.g4
View File

@ -25,7 +25,7 @@ lexer grammar UriLexer;
//;============================================================================== //;==============================================================================
QM : '?' -> pushMode(MODE_QUERY); //first query parameter QM : '?' -> pushMode(MODE_QUERY); //first query parameter
AMP : '&' -> pushMode(MODE_QUERY); //more query parameters AMP : '&' -> pushMode(MODE_QUERY); //more query parameters
STRING : '\'' -> more, pushMode(MODE_STRING); //reads up to next single ' STRING : '\'' -> more, pushMode(MODE_STRING) ; //reads up to next single '
QUOTATION_MARK : '\u0022' -> more, pushMode(MODE_JSON_STRING); //reads up to next unescaped " QUOTATION_MARK : '\u0022' -> more, pushMode(MODE_JSON_STRING); //reads up to next unescaped "
SEARCH_INLINE : '$search' -> pushMode(MODE_SYSTEM_QUERY_SEARCH); // SEARCH_INLINE : '$search' -> pushMode(MODE_SYSTEM_QUERY_SEARCH); //
FRAGMENT : '#' -> pushMode(MODE_FRAGMENT); // FRAGMENT : '#' -> pushMode(MODE_FRAGMENT); //
@ -311,7 +311,7 @@ mode MODE_FRAGMENT;
// character. // character.
//;============================================================================== //;==============================================================================
REST_F : ~('\r'|'\n')* -> type(REST), popMode; REST_F : ~('\r'|'\n')* -> type(REST), popMode;
//;============================================================================== //;==============================================================================
mode MODE_SYSTEM_QUERY_REST; mode MODE_SYSTEM_QUERY_REST;
@ -319,33 +319,35 @@ mode MODE_SYSTEM_QUERY_REST;
// character. // character.
//;============================================================================== //;==============================================================================
AMP_sqr : '&' -> type(AMP), popMode; AMP_sqr : '&' -> type(AMP), popMode;
FRAGMENT_sqr : '#' -> type(FRAGMENT), popMode; FRAGMENT_sqr : '#' -> type(FRAGMENT), popMode;
EQ_sqr : '=' -> type(EQ); EQ_sqr : '=' -> type(EQ);
REST : ~[&#=] ~[&#]*; REST : ~[&#=] ~[&#]*;
ERROR_CHARACTER_sqmr : .;
//;============================================================================== //;==============================================================================
mode MODE_SYSTEM_QUERY_SEARCH; mode MODE_SYSTEM_QUERY_SEARCH;
//;============================================================================== //;==============================================================================
NOT_sqc : 'NOT' -> type(NOT); NOT_sqc : 'NOT' -> type(NOT);
AND_sqc : 'AND' -> type(AND); AND_sqc : 'AND' -> type(AND);
OR_sqc : 'OR' -> type(OR); OR_sqc : 'OR' -> type(OR);
EQ_sqc : '=' -> type(EQ); EQ_sqc : '=' -> type(EQ);
fragment WS_sqc : ( ' ' | '\u0009'); fragment WS_sqc : ( ' ' | '\u0009');
WSP_sqc : WS_sqc+ -> type(WSP); WSP_sqc : WS_sqc+ -> type(WSP);
QUOTATION_MARK_sqc : '\u0022'; QUOTATION_MARK_sqc : '\u0022';
SEARCHWORD : ('a'..'z'|'A'..'Z')+; SEARCHWORD : ('a'..'z'|'A'..'Z')+;
SEARCHPHRASE : QUOTATION_MARK_sqc ~["]* QUOTATION_MARK_sqc; SEARCHPHRASE : QUOTATION_MARK_sqc ~["]* QUOTATION_MARK_sqc;
// Follow Set // Follow Set
CLOSE_qs : ')' -> popMode, type(CLOSE); CLOSE_qs : ')' -> popMode, type(CLOSE);
SEMI_qs : ';' -> popMode, type(SEMI); SEMI_qs : ';' -> popMode, type(SEMI);
AMP_qs : '&' -> popMode, type(AMP); AMP_qs : '&' -> popMode, type(AMP);
ERROR_CHARACTER_sqms : .;
//;============================================================================== //;==============================================================================
mode MODE_STRING; mode MODE_STRING;
@ -353,7 +355,8 @@ mode MODE_STRING;
// Any "'" characters inside a string are expressed as double "''". // Any "'" characters inside a string are expressed as double "''".
//;============================================================================== //;==============================================================================
STRING_s : ('\'\'' | ~[\u0027] )* '\'' -> type(STRING), popMode; STRING_s : ('\'\'' | ~[\u0027] )* '\'' -> type(STRING), popMode;
ERROR_CHARACTER_sm : EOF | .;
//;============================================================================== //;==============================================================================
mode MODE_JSON_STRING; mode MODE_JSON_STRING;
@ -361,7 +364,8 @@ mode MODE_JSON_STRING;
// Any """ characters inside a string are escaped with "\". // Any """ characters inside a string are escaped with "\".
//;============================================================================== //;==============================================================================
STRING_IN_JSON : ('\\"' | ~[\u0022] )* '"' -> popMode; STRING_IN_JSON : ('\\"' | ~[\u0022] )* '"' -> popMode;
ERROR_CHARACTER_jsm : EOF | .;
//;============================================================================== //;==============================================================================
mode MODE_ODATA_GEO; mode MODE_ODATA_GEO;
@ -412,3 +416,5 @@ POLYGON : P_ O_ L_ Y_ G_ O_ N_ ;
SRID : S_ R_ I_ D_; SRID : S_ R_ I_ D_;
SQUOTE : '\'' -> popMode; SQUOTE : '\'' -> popMode;
ERROR_CHARACTER_g : .;

21
lib/server-test/src/test/java/org/apache/olingo/server/core/uri/antlr/TestFullResourcePath.java
View File

@ -1013,6 +1013,25 @@ public class TestFullResourcePath {
.isExSemantic(UriParserSemanticException.MessageKeys.RESOURCE_NOT_FOUND); .isExSemantic(UriParserSemanticException.MessageKeys.RESOURCE_NOT_FOUND);
} }
@Test
public void runResourcePathWithApostrophe() {
// TODO Currently "'" is not allowed in OData identifiers, but the specification allows this character (Unicode Cf)
testUri.runEx("ESAllPrim'").isExSyntax(UriParserSyntaxException.MessageKeys.SYNTAX);
testUri.runEx("ESAllPrim'InvalidStuff").isExSyntax(UriParserSyntaxException.MessageKeys.SYNTAX);
testUri.runEx("ESAllPrim", "$filter=PropertyInt16' eq 0")
.isExSyntax(UriParserSyntaxException.MessageKeys.SYNTAX);
testUri.runEx("ESAllPrim", "$filter=PropertyInt16 eq' 0")
.isExSyntax(UriParserSyntaxException.MessageKeys.SYNTAX);
testUri.runEx("ESAllPrim", "$filter=PropertyInt16 eq 0'")
.isExSyntax(UriParserSyntaxException.MessageKeys.SYNTAX);
testUri.runEx("ESAllPrim", "$filter=PropertyInt16 eq 'dsd''")
.isExSyntax(UriParserSyntaxException.MessageKeys.SYNTAX);
}
@Test @Test
public void runEsNameCast() throws Exception { public void runEsNameCast() throws Exception {
testUri.run("ESTwoPrim/olingo.odata.test1.ETBase") testUri.run("ESTwoPrim/olingo.odata.test1.ETBase")
@ -5351,7 +5370,7 @@ public class TestFullResourcePath {
+ "(PropertyInt=1,PropertyString='2')") + "(PropertyInt=1,PropertyString='2')")
.isExSemantic(MessageKeys.WRONG_NUMBER_OF_KEY_PROPERTIES); .isExSemantic(MessageKeys.WRONG_NUMBER_OF_KEY_PROPERTIES);
} }
@Test @Test
public void startElementsInsteadOfNavigationProperties() { public void startElementsInsteadOfNavigationProperties() {
testUri.runEx("ESAllPrim(0)/ESAllPrim(0)/ESAllPrim(0)").isExSemantic(MessageKeys.PROPERTY_NOT_IN_TYPE); testUri.runEx("ESAllPrim(0)/ESAllPrim(0)/ESAllPrim(0)").isExSemantic(MessageKeys.PROPERTY_NOT_IN_TYPE);