Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

box.com invoice example: path of /rest/upload incorrect #40

Open
mathiasconradt opened this issue Jan 2, 2018 · 2 comments
Open

box.com invoice example: path of /rest/upload incorrect #40

mathiasconradt opened this issue Jan 2, 2018 · 2 comments

Comments

@mathiasconradt
Copy link

mathiasconradt commented Jan 2, 2018

There is a duplicate /rest in the path for /rest/upload.

(I am not actually sure if this is an issue or not, but still wondering why that is. It looks a bit uncommon.)

https://github.com/camunda-consulting/code/blob/0d1ebd05825adb65092a5c59375f5e9398378dae/snippets/ecm-integrations/invoice-box/src/main/webapp/forms/upload-new.html#L46

@mathiasconradt
Copy link
Author

mathiasconradt commented Jan 2, 2018

Hm, actually I find this double /rest/rest/... path in multiple source files / html forms. Is it intentionally? But why?

UploadFacade.java:

@Path("/rest")
public class UploadFacade {

RestApplication.java

@ApplicationPath("/rest")
public class RestApplication extends Application {
}

In any case, the UploadFacade does not seem to be properly registered.

In both cases, the URLs

GET http://localhost:8080/camunda-invoice-box/rest/rest/embedLink
GET http://localhost:8080/camunda-invoice-box/rest/embedLink

show a 404 error.

On Camunda 7.8.0 CE with Tomcat.

@felix-mueller
Copy link
Member

Even though it's weird, twice /rest works on Wildfly. Probably could be fixed in the example.
Do you want to create a PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants