Skip to content

Commit

Permalink
🐛 Correctly handle nested configs
Browse files Browse the repository at this point in the history
Apologies for the extensiveness of this change. The configuration
handling likely needs a refactor to make it easier to support future
modification.

 - Add Apache Commons Configuration2

 - Adapt configuration2's INIConfiguration to support AWS quirks

 - Remove Ini4j (unmaintained, has park page for website)

 - Refactor file handling code

Resolves oktadev#141
  • Loading branch information
AlainODea committed Aug 19, 2018
1 parent 12c8e2d commit 87f90ac
Show file tree
Hide file tree
Showing 14 changed files with 1,437 additions and 366 deletions.
17 changes: 12 additions & 5 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,11 @@
<artifactId>json</artifactId>
<version>${json.version}</version>
</dependency>
<dependency>
<groupId>org.apache.commons</groupId>
<artifactId>commons-configuration2</artifactId>
<version>2.3</version>
</dependency>
<dependency>
<groupId>org.apache.logging.log4j</groupId>
<artifactId>log4j-api</artifactId>
Expand All @@ -85,11 +90,6 @@
<artifactId>log4j-core</artifactId>
<version>${log4j.version}</version>
</dependency>
<dependency>
<groupId>org.ini4j</groupId>
<artifactId>ini4j</artifactId>
<version>${ini4j.version}</version>
</dependency>
<dependency>
<groupId>org.jsoup</groupId>
<artifactId>jsoup</artifactId>
Expand Down Expand Up @@ -138,6 +138,13 @@
<version>1.0.0</version>
<scope>test</scope>
</dependency>
<!-- Dynamic dependency of commons-configuration -->
<dependency>
<groupId>commons-beanutils</groupId>
<artifactId>commons-beanutils</artifactId>
<version>1.9.3</version>
<scope>runtime</scope>
</dependency>
</dependencies>

<distributionManagement>
Expand Down
Loading

0 comments on commit 87f90ac

Please sign in to comment.