Opened 4 years ago

Closed 4 years ago

#19727 closed defect (fixed)

correct exitlist

Reported by: iwakeh Owned by:
Priority: Medium Milestone: CollecTor 1.0.0
Component: Metrics/CollecTor Version:
Severity: Normal Keywords: ctip
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


Make exitlists url configurable and correct current setting:

The current setting is wrong and should be changed to

Child Tickets

Change History (5)

comment:1 Changed 4 years ago by karsten

You're right, I still have that non-staged change on the CollecTor server, because that URL changed a few times in the past and I didn't know how permanent the current change would be, and then I forgot. Oops. Do you have a patch?

comment:2 Changed 4 years ago by iwakeh

Status: newneeds_review

Please review the implementation.

Clean compile, tests pass, coverage stays the same, no checkstyle complaints added.

This is a good example for adding a property setting with a new class type.

comment:3 Changed 4 years ago by karsten

Patch looks good, but I'm having trouble testing it. It's probably something unrelated, but I don't know what. Do you have an idea?

~/src/collector$ java -jar collector-0.9.0-dev.jar
Error: A JNI error has occurred, please check your installation and try again
Exception in thread "main" java.lang.SecurityException: Invalid signature file digest for Manifest main attributes
	at java.util.jar.JarVerifier.processEntry(
	at java.util.jar.JarVerifier.update(
	at java.util.jar.JarFile.initializeVerifier(
	at java.util.jar.JarFile.getInputStream(
	at sun.misc.URLClassPath$JarLoader$2.getInputStream(
	at sun.misc.Resource.cachedInputStream(
	at sun.misc.Resource.getByteBuffer(
	at Method)
	at java.lang.ClassLoader.loadClass(
	at sun.misc.Launcher$AppClassLoader.loadClass(
	at java.lang.ClassLoader.loadClass(
	at sun.launcher.LauncherHelper.checkAndLoadMain(

comment:4 Changed 4 years ago by iwakeh

That's probably what I patched here.

comment:5 Changed 4 years ago by karsten

Resolution: fixed
Status: needs_reviewclosed

Yes, that patch makes it work just fine. I pushed your task-19727 branch but didn't push that commit 1426040 yet which we should merge soon as part of the other branch. Closing. Thanks!

Note: See TracTickets for help on using tickets.