Site changes for LDAP API 2.0

2018-07-25 Thread Emmanuel Lécharny
Hi guys, I'm changing a few things on the web site for the LDAP API v2. It's not completed yet, so it's only visible on staging (directory.staging.apache.org) Please do not publish the site until I'm done ! Thanks !

[GitHub] smoyer64 commented on issue #5: Use maven dist management from Apache parent pom

2018-07-25 Thread GitBox
smoyer64 commented on issue #5: Use maven dist management from Apache parent pom URL: https://github.com/apache/directory-scimple/pull/5#issuecomment-407923869 Yes ... I didn't mean to imply that what we document not conform to the Apache release rules (I read through that thread on the dev

[GitHub] smoyer64 closed pull request #4: Add Jacoco Maven plugin to record test coverage

2018-07-25 Thread GitBox
smoyer64 closed pull request #4: Add Jacoco Maven plugin to record test coverage URL: https://github.com/apache/directory-scimple/pull/4 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] smoyer64 commented on issue #4: Add Jacoco Maven plugin to record test coverage

2018-07-25 Thread GitBox
smoyer64 commented on issue #4: Add Jacoco Maven plugin to record test coverage URL: https://github.com/apache/directory-scimple/pull/4#issuecomment-407896167 NP ... I've been there too This is an automated message from the Ap

[GitHub] smoyer64 commented on issue #9: WIP: Updating NOTICE file based on Apache ref doc

2018-07-25 Thread GitBox
smoyer64 commented on issue #9: WIP: Updating NOTICE file based on Apache ref doc URL: https://github.com/apache/directory-scimple/pull/9#issuecomment-407895577 @ussmith This is marked WIP @bdemers Should we revert? This i

[GitHub] bdemers commented on issue #4: Add Jacoco Maven plugin to record test coverage

2018-07-25 Thread GitBox
bdemers commented on issue #4: Add Jacoco Maven plugin to record test coverage URL: https://github.com/apache/directory-scimple/pull/4#issuecomment-407885201 @smoyer64 got it, I'm juggling a bunch of small branches 😄 This is

[GitHub] bdemers commented on issue #4: Add Jacoco Maven plugin to record test coverage

2018-07-25 Thread GitBox
bdemers commented on issue #4: Add Jacoco Maven plugin to record test coverage URL: https://github.com/apache/directory-scimple/pull/4#issuecomment-407883299 @smoyer64 sorry about that, I'll clear my local repo fix and verify (i'm trying to setup a PR CI job to catch this type of thing)

[GitHub] ussmith closed pull request #9: WIP: Updating NOTICE file based on Apache ref doc

2018-07-25 Thread GitBox
ussmith closed pull request #9: WIP: Updating NOTICE file based on Apache ref doc URL: https://github.com/apache/directory-scimple/pull/9 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[GitHub] bdemers commented on issue #9: WIP: Updating NOTICE file based on Apache ref doc

2018-07-25 Thread GitBox
bdemers commented on issue #9: WIP: Updating NOTICE file based on Apache ref doc URL: https://github.com/apache/directory-scimple/pull/9#issuecomment-407879865 build not triggering... This is an automated message from the Apac

[GitHub] asfgit commented on issue #9: Updating NOTICE file based on Apache ref doc

2018-07-25 Thread GitBox
asfgit commented on issue #9: Updating NOTICE file based on Apache ref doc URL: https://github.com/apache/directory-scimple/pull/9#issuecomment-407878732 Can one of the admins verify this patch? This is an automated message fr

[GitHub] bdemers commented on issue #9: Updating NOTICE file based on Apache ref doc

2018-07-25 Thread GitBox
bdemers commented on issue #9: Updating NOTICE file based on Apache ref doc URL: https://github.com/apache/directory-scimple/pull/9#issuecomment-407878852 (mostly pushing this to see if the CI trigger will work) This is an aut

[GitHub] bdemers opened a new pull request #9: Updating NOTICE file based on Apache ref doc

2018-07-25 Thread GitBox
bdemers opened a new pull request #9: Updating NOTICE file based on Apache ref doc URL: https://github.com/apache/directory-scimple/pull/9 https://www.apache.org/dev/licensing-howto.html This is an automated message from the

[GitHub] smoyer64 commented on issue #8: Reducing default log level in BaseResourceTypeResourceImpl

2018-07-25 Thread GitBox
smoyer64 commented on issue #8: Reducing default log level in BaseResourceTypeResourceImpl URL: https://github.com/apache/directory-scimple/pull/8#issuecomment-407878377 Strange that this file had both a ``@Slf4j`` Lombok annotation and a statically declared ``Logger``. I think the norm i

[GitHub] asfgit commented on issue #4: Add Jacoco Maven plugin to record test coverage

2018-07-25 Thread GitBox
asfgit commented on issue #4: Add Jacoco Maven plugin to record test coverage URL: https://github.com/apache/directory-scimple/pull/4#issuecomment-407878106 Can one of the admins verify this patch? This is an automated message

[GitHub] smoyer64 commented on issue #4: Add Jacoco Maven plugin to record test coverage

2018-07-25 Thread GitBox
smoyer64 commented on issue #4: Add Jacoco Maven plugin to record test coverage URL: https://github.com/apache/directory-scimple/pull/4#issuecomment-407877377 There are still a bunch of references to ``2.0-SNAPSHOT`` in the two changed files. I was expecting a rebase to fix them but now I

[GitHub] bdemers commented on issue #4: Add Jacoco Maven plugin to record test coverage

2018-07-25 Thread GitBox
bdemers commented on issue #4: Add Jacoco Maven plugin to record test coverage URL: https://github.com/apache/directory-scimple/pull/4#issuecomment-407875531 @smoyer64 this should merge clean now This is an automated messa

Re: Added directory-scimple to builds.a.o

2018-07-25 Thread Brian Demers
It's blue! On Wed, Jul 25, 2018 at 10:50 AM, Brian Demers wrote: > > Almost there :) > > Fixes OWASP dep scan issue: > https://github.com/apache/directory-scimple/pull/6 > > On Tue, Jul 24, 2018 at 1:59 PM, Brian Demers wrote: > >> > * I didn't see directory-scimple listed on https://git.apache

[GitHub] bdemers opened a new pull request #8: Reducing default log level in BaseResourceTypeResourceImpl

2018-07-25 Thread GitBox
bdemers opened a new pull request #8: Reducing default log level in BaseResourceTypeResourceImpl URL: https://github.com/apache/directory-scimple/pull/8 Removing the duplicate log declaring by lombok (there was already a logger defined in this class) Changed log level from `error` to `d

[GitHub] asfgit commented on issue #8: Reducing default log level in BaseResourceTypeResourceImpl

2018-07-25 Thread GitBox
asfgit commented on issue #8: Reducing default log level in BaseResourceTypeResourceImpl URL: https://github.com/apache/directory-scimple/pull/8#issuecomment-407871709 Can one of the admins verify this patch? This is an autom

[GitHub] ussmith closed pull request #7: Allow Provider implementations to handle JAX-RS exceptions by default

2018-07-25 Thread GitBox
ussmith closed pull request #7: Allow Provider implementations to handle JAX-RS exceptions by default URL: https://github.com/apache/directory-scimple/pull/7 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of pro

[GitHub] bdemers opened a new pull request #7: Allow Provider implementations to handle JAX-RS exceptions by default

2018-07-25 Thread GitBox
bdemers opened a new pull request #7: Allow Provider implementations to handle JAX-RS exceptions by default URL: https://github.com/apache/directory-scimple/pull/7 WebApplicationException will be handled automatically by ErrorMessageViolationExceptionMapper by default

[GitHub] smoyer64 closed pull request #6: Update OWASP dep scan configuration

2018-07-25 Thread GitBox
smoyer64 closed pull request #6: Update OWASP dep scan configuration URL: https://github.com/apache/directory-scimple/pull/6 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pul

Re: Added directory-scimple to builds.a.o

2018-07-25 Thread Brian Demers
Almost there :) Fixes OWASP dep scan issue: https://github.com/apache/directory-scimple/pull/6 On Tue, Jul 24, 2018 at 1:59 PM, Brian Demers wrote: > > * I didn't see directory-scimple listed on https://git.apache.org/, so I >> > used the GitHub URL. What is the canonical URL for this repo? >>

[GitHub] bdemers commented on issue #2: WIP: Add in memory example - Port PennState in memory server example to new Apache structure

2018-07-25 Thread GitBox
bdemers commented on issue #2: WIP: Add in memory example - Port PennState in memory server example to new Apache structure URL: https://github.com/apache/directory-scimple/pull/2#issuecomment-407807963 Ha, I had only updated the commons-jaxrs dep version, not the GAV. Fixed :)

[GitHub] bdemers commented on issue #2: WIP: Add in memory example - Port PennState in memory server example to new Apache structure

2018-07-25 Thread GitBox
bdemers commented on issue #2: WIP: Add in memory example - Port PennState in memory server example to new Apache structure URL: https://github.com/apache/directory-scimple/pull/2#issuecomment-407798921 I thought i had already rebased this... 🤔

[jira] [Created] (DIR-331) Remove usage of restfuse from SCIMple (the project is dead)

2018-07-25 Thread Brian Demers (JIRA)
Brian Demers created DIR-331: Summary: Remove usage of restfuse from SCIMple (the project is dead) Key: DIR-331 URL: https://issues.apache.org/jira/browse/DIR-331 Project: Directory Issue Type:

[GitHub] bdemers opened a new pull request #6: Update OWASP dep scan configuration

2018-07-25 Thread GitBox
bdemers opened a new pull request #6: Update OWASP dep scan configuration URL: https://github.com/apache/directory-scimple/pull/6 Use the `check` goal instead of `aggregate` which can only be run post build (or as part of a site/report build) Updated dependencies with scan: * comm