Skip to content
Snippets Groups Projects

Issue resolving Github raw content site in cluster

    • Closed Issue created by Martin Lowe @malowe

    With the Gitlab token resolved, another problem arose in the cluster, where the host raw.githubusercontent.com could not be resolved. This is an unprotected endpoint that is publicly available, so I'm not sure why the API can't resolve it. I'll be looking to reproduce it today.

    Designs

      Child items ...

      2. Activity

      • Martin Lowe added statewip label

        added statewip label

      • Martin Lowe assigned to @malowe

        assigned to @malowe

      • Martin Lowe
        Martin Lowe @malowe ·
        Author Maintainer

        This is related to eclipsefdn/helpdesk#912 (closed). The fix is to add a GH PAT and do authenticated calls to fetch content instead of unauthenticated calls.

      • Martin Lowe assigned to @zacharysabourin

        assigned to @zacharysabourin

      • Martin Lowe unassigned @malowe

        unassigned @malowe

      • Martin Lowe added statetodo label and removed statewip label

        added statetodo label and removed statewip label

      • Zachary Sabourin added statewip label and removed statetodo label

        added statewip label and removed statetodo label

      • Zachary Sabourin added #17 (closed) as child task

        added #17 (closed) as child task

      • Zachary Sabourin added #18 (closed) as child task

        added #18 (closed) as child task

      • Zachary Sabourin added #19 (closed) as child task

        added #19 (closed) as child task

      • Zachary Sabourin added #20 (closed) as child task

        added #20 (closed) as child task

      • Zachary Sabourin added #21 (closed) as child task

        added #21 (closed) as child task

      • Zachary Sabourin added #22 (closed) as child task

        added #22 (closed) as child task

      • Zachary Sabourin mentioned in merge request !20 (merged)

        mentioned in merge request !20 (merged)

      • Zachary Sabourin
        Zachary Sabourin @zacharysabourin ·
        Contributor

        The initial assessment of tasks became obsolete once it became clear that we could still get raw file contents from api.github.com

        All that needed to change was the GH URL and some modification to the GH rest client to use new params and auth token

      • Zachary Sabourin added statereview label and removed statewip label

        added statereview label and removed statewip label

      • Martin Lowe mentioned in commit 025b5ef6

        mentioned in commit 025b5ef6

      • Martin Lowe closed with merge request !20 (merged)

        closed with merge request !20 (merged)

      • Martin Lowe reopened

        reopened

      • Martin Lowe
        Martin Lowe @malowe ·
        Author Maintainer

        This issue is not yet resolved, as we need to get an access token to enable this on production/staging

      • Martin Lowe
        Martin Lowe @malowe ·
        Author Maintainer

        I've pushed this to prod with the token and its having errors while trying to read the incoming JSON data:

        Caused by: java.lang.IllegalArgumentException: RESTEASY003340: Failure parsing MediaType string: application/json,application/vnd.github.VERSION.raw; charset=utf-8
	at org.jboss.resteasy.plugins.delegates.MediaTypeHeaderDelegate.internalParse(MediaTypeHeaderDelegate.java:118)
	at org.jboss.resteasy.plugins.delegates.MediaTypeHeaderDelegate.parse(MediaTypeHeaderDelegate.java:69)
	at org.jboss.resteasy.plugins.delegates.MediaTypeHeaderDelegate.fromString(MediaTypeHeaderDelegate.java:33)
	at org.jboss.resteasy.plugins.delegates.MediaTypeHeaderDelegate.fromString(MediaTypeHeaderDelegate.java:19)
	at javax.ws.rs.core.MediaType.valueOf(MediaType.java:172)
	at org.jboss.resteasy.specimpl.AbstractBuiltResponse.getMediaType(AbstractBuiltResponse.java:342)
	at org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.extractResult(ClientInvocation.java:207)
	at org.jboss.resteasy.client.jaxrs.internal.proxy.extractors.BodyEntityExtractor.extractEntity(BodyEntityExtractor.java:64)
	at org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invokeSync(ClientInvoker.java:154)
	at org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invoke(ClientInvoker.java:115)
	at org.jboss.resteasy.client.jaxrs.internal.proxy.ClientProxy.invoke(ClientProxy.java:76)
	at com.sun.proxy.$Proxy32.getCveDetails(Unknown Source)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at org.jboss.resteasy.microprofile.client.ProxyInvocationHandler.invoke(ProxyInvocationHandler.java:146)
	at com.sun.proxy.$Proxy35.getCveDetails(Unknown Source)
	at org.eclipsefoundation.cve.service.impl.DefaultCveService.loadCveProjectData(DefaultCveService.java:174)
	... 10 more

2022-11-24 15:05:41,968 ERROR [io.qua.run.Application] (main) Failed to start application (with profile prod): java.lang.IllegalArgumentException: RESTEASY003340: Failure parsing MediaType string: application/json,application/vnd.github.VERSION.raw; charset=utf-8
	at org.jboss.resteasy.plugins.delegates.MediaTypeHeaderDelegate.internalParse(MediaTypeHeaderDelegate.java:118)
	at org.jboss.resteasy.plugins.delegates.MediaTypeHeaderDelegate.parse(MediaTypeHeaderDelegate.java:69)
	at org.jboss.resteasy.plugins.delegates.MediaTypeHeaderDelegate.fromString(MediaTypeHeaderDelegate.java:33)
	at org.jboss.resteasy.plugins.delegates.MediaTypeHeaderDelegate.fromString(MediaTypeHeaderDelegate.java:19)
	at javax.ws.rs.core.MediaType.valueOf(MediaType.java:172)
	at org.jboss.resteasy.specimpl.AbstractBuiltResponse.getMediaType(AbstractBuiltResponse.java:342)
	at org.jboss.resteasy.client.jaxrs.internal.ClientInvocation.extractResult(ClientInvocation.java:207)
	at org.jboss.resteasy.client.jaxrs.internal.proxy.extractors.BodyEntityExtractor.extractEntity(BodyEntityExtractor.java:64)
	at org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invokeSync(ClientInvoker.java:154)
	at org.jboss.resteasy.client.jaxrs.internal.proxy.ClientInvoker.invoke(ClientInvoker.java:115)
	at org.jboss.resteasy.client.jaxrs.internal.proxy.ClientProxy.invoke(ClientProxy.java:76)
	at com.sun.proxy.$Proxy32.getCveDetails(Unknown Source)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)
	at org.jboss.resteasy.microprofile.client.ProxyInvocationHandler.invoke(ProxyInvocationHandler.java:146)
	at com.sun.proxy.$Proxy35.getCveDetails(Unknown Source)
	at org.eclipsefoundation.cve.service.impl.DefaultCveService.loadCveProjectData(DefaultCveService.java:174)
	at com.github.benmanes.caffeine.cache.CacheLoader.lambda$asyncLoad$0(CacheLoader.java:104)
	at java.base/java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1700)
	at io.smallrye.context.impl.wrappers.SlowContextualRunnable.run(SlowContextualRunnable.java:19)
	at io.quarkus.vertx.core.runtime.VertxCoreRecorder$14.runWith(VertxCoreRecorder.java:555)
	at org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2449)
	at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1478)
	at org.jboss.threads.DelegatingRunnable.run(DelegatingRunnable.java:29)
	at org.jboss.threads.ThreadLocalResettingRunnable.run(ThreadLocalResettingRunnable.java:29)
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
	at java.base/java.lang.Thread.run(Thread.java:829)
      • Martin Lowe added statewip label and removed statereview label

        added statewip label and removed statereview label

      • Zachary Sabourin mentioned in merge request !22 (merged)

        mentioned in merge request !22 (merged)

      • Zachary Sabourin added statereview label and removed statewip label

        added statereview label and removed statewip label

      • Martin Lowe added statedone label and removed statereview label

        added statedone label and removed statereview label

      • Martin Lowe closed

        closed

      Please register or sign in to reply

      Copyright © Eclipse Foundation, Inc. All Rights Reserved.     Privacy Policy | Terms of Use | Copyright Agent