10 Nov, 2014
1 commit
-
git-svn-id: https://svn.forgerock.org/openig/trunk@696 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
07 Nov, 2014
9 commits
-
Wrong charset used in Entity class - Added static UTF_8 Charset. - Replaced newDecodedContentReader(null) by newDecodedContentReader(UTF_8) in entity#getJson. git-svn-id: https://svn.forgerock.org/openig/trunk@695 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
Without this change, the OAuth 2.0 Client Filter triggers the retrieval of the user info resource for each request that is intercepted. It's a problem for Identity Providers such as Google that have an allowed quota of request/sec, because of the sudden burst of user info request. This is even worse than that because sometimes, the intercepted request may not even need to use theses information (think of OpenIG intercepting an image served by the protected application and returned as-is) ... This fix includes both a user-info resources caching for a few seconds (the time for all requests to load a web page to be executed) and a lazy loading of the resource (triggered the first time a downstream filter/handler access the `user_info` structure). By default, resources are kept for 20 seconds after the first access. Like for OAuth2ResourceServerFilter, you can disable that cache with `"cacheExpiration": "disabled"` in the configuration. git-svn-id: https://svn.forgerock.org/openig/trunk@694 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
This takes advantage of the new zero-length Duration support. If the configuration express `zero`, `disabled` or any `0 <timeunit>`, the access token cache will be disabled. git-svn-id: https://svn.forgerock.org/openig/trunk@693 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
That will be useful to support deactivation marker values when configuring cache timeouts. This fix support both `zero` and `disabled` as zero-length Duration markers. When the parsed String value also represents a zero-length duration (like `0 days and 0 ms`), the special ZERO marker duration is returned. Updated documentation as well. git-svn-id: https://svn.forgerock.org/openig/trunk@692 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@691 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@690 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
In order to facilitate OPENIG-370 (log simplification), we now expose the Name of the log source to the Logger. This has a number of implications: * `LogEntry.source` has been splited into the `source` and `type` attributes. source being the original Name and type being the type of the entry (like `log`, `started`, `elapsed`, ...). Log messages have the `log` type and exception's messages have the `throwable` type. * Logger is no more a LogSink implementation * LogEntry has been adapted to directly use Logger instead of LogSink (this way it can benefit of the Logger.source attribute's value automatically) * Logger do not "rebase" the source name anymore: can't do that since source is not a String anymore. * That solves the source duplication issue of OPENIG-370 LogSink interface has been adapted to have a source Name instead of a source String when trying to determine if something is legible. LogSink implementations (ConsoleLogSink and FileLogSink) have been changed to keep an output as close as possible as what we have previously (don't want to work on OPEN-370 yet). git-svn-id: https://svn.forgerock.org/openig/trunk@689 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
A Name is a recursive object based on a leaf name and a parent Name. Each Heap as a unique Name used to scope names of the managed heap objects. Each heap object has a unique Name whose parent is the container Heap. The Heaplet interface has been updated to give the Name instead of just a String. The name of each created heap is based on the resource name. git-svn-id: https://svn.forgerock.org/openig/trunk@688 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@687 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
06 Nov, 2014
2 commits
-
git-svn-id: https://svn.forgerock.org/openig/trunk@686 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
Since forgerock-parent 1.2.6, we need to specify that property in addition to `maven.compiler.target`. IntelliJ can now import the project properly (using Java 6 style instead of Java 5 for code). git-svn-id: https://svn.forgerock.org/openig/trunk@685 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
05 Nov, 2014
5 commits
-
git-svn-id: https://svn.forgerock.org/openig/trunk@684 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@683 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
* OAuth2TokenException refactored to only have a message field * Do not re-print chained exception's message * Better error messages git-svn-id: https://svn.forgerock.org/openig/trunk@682 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@681 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@680 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
04 Nov, 2014
3 commits
-
git-svn-id: https://svn.forgerock.org/openig/trunk@679 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@678 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
When capturing the exchange's content, we're using JsonOutput with a Map view of the Exchange. It happens that this view, because of the FieldMap stuff, only includes exchange's public fields. It looks like this is not an issue for Expressions. We'll fix all of theses public properties at once git-svn-id: https://svn.forgerock.org/openig/trunk@677 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
03 Nov, 2014
2 commits
-
* `preparedStatement` attribute is required * `parameters` could be optional if no placeholders are specified Used the as-list-of construct to init the parameters' expression's value. Synched the doc git-svn-id: https://svn.forgerock.org/openig/trunk@676 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@675 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
31 Oct, 2014
2 commits
-
The JDBC spec states that PreparedStatement created through Connection.prepareStatement(String) have the FORWARD_ONLY type. A forward only statement may not implement the first() method (does not make sense to go back to the first row when you can only move forward). So, it's safer to use ResultSet.next(). git-svn-id: https://svn.forgerock.org/openig/trunk@674 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
Trying to fill a non-existent prepared-statement placeholder results in a SQLException (`Invalid column index` for Oracle, some other for MySQL, ...), so we simply avoid theses errors by ignoring extra parameters. We also output meaningful (I hope) warnings for both extra parameters and un-valued placeholders. git-svn-id: https://svn.forgerock.org/openig/trunk@673 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
30 Oct, 2014
6 commits
-
The CaptureDecorator can now be configured to additionally capture the content of the Exchange being intercepted. If specified, it will print the full content of the Exchange instance, excluding the `response` and the `request` since they should be captured by another mean. The output contains a deep view of the exchange obtained through introspection of the class structure (this is done by the Groovy JsonOutput class). It appears that this class doesn't support cycles very well, so, depending on the content of the exchange, it may throw exceptions. By default we're doing our best to remove known cycles. By default (in the system provided `capture` decorator), this option is disabled since it adds a lot of cruft in the logs. git-svn-id: https://svn.forgerock.org/openig/trunk@672 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@671 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
…nfigured in the config.json Instead of creating manually the HttpClient instance and giving it the (also manually) created TemporaryStorage instance, we now declare an equivalent object declaration in the heap, letting the HttpClient heaplet resolves properly its dependencies. git-svn-id: https://svn.forgerock.org/openig/trunk@670 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@669 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@668 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
This aligns with r611. Thanks to Guillaume for pointing out what I'd missed. git-svn-id: https://svn.forgerock.org/openig/trunk@667 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
29 Oct, 2014
1 commit
-
Although the JSON pointer is sufficient to guarantee unicity of a name within the scope of a heap, it makes it hard for reader to link that name to the configuration object they provided. This patch supports both improvements: * if a `name` attribute is provided in the declaration, use it as-is * if not, prepend the `type` attribute value to the pointer String to help identification This patch also updates the logs samples provided in the documentation. git-svn-id: https://svn.forgerock.org/openig/trunk@666 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
28 Oct, 2014
1 commit
-
git-svn-id: https://svn.forgerock.org/openig/trunk@665 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
24 Oct, 2014
7 commits
-
git-svn-id: https://svn.forgerock.org/openig/trunk@664 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@663 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@662 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@661 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@660 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
Beginning with the removal of routes guarantee that we won't remove a route that was just added or modified. git-svn-id: https://svn.forgerock.org/openig/trunk@659 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
-
git-svn-id: https://svn.forgerock.org/openig/trunk@658 dbb9e58e-28e6-4ce0-90e8-f11d9605b710
23 Oct, 2014
1 commit
-
Exception's messages does not have to include the message of their parent anymore. When a route failed to load for any reason, all the chained exception's messages are logged as errors up to the root cause, and the full stack trace is logged at the DEBUG level. Notice that the LogSink you have to configure is the one of the Router itself (in the JSON configuration where the Router handler is declared), configuring the LogSing of the failed route configuration will not help. git-svn-id: https://svn.forgerock.org/openig/trunk@657 dbb9e58e-28e6-4ce0-90e8-f11d9605b710