caelum / vraptor-quartzjob

A simple quartz job
3 stars 11 forks source link

Bug na 4.0.7 #15

Open fredericocostachaves opened 7 years ago

fredericocostachaves commented 7 years ago

Mais precisamente em: String url = (env.get("host") + ctx.getContextPath() + "/jobs/configure").replace("https", "http"); Esse ctx.getContextPath() não deve existir e o atributo host deve conter o nome da aplicação. Caso contrário na hora de chamar o job, o nome da aplicação repete...

tiagojco commented 7 years ago

Essa atribuição ocorre na linha 51 do QuartzConfigurator, onde ele assume o valor da expressão env.get("host") não possui o contextPath, e adiciona ele programaticamente.

Só que na linha 80 do QuartzScheduler ele assume que o valor host já possui o contextPath.

Concordo com @fredericocostachaves, esse ctx.getContextPath() não deveria existir, e o desenvolvedor não tem como "contornar" esse problema, pois:

Criei o PR #16 para corrigir isso

andersonscherrer commented 6 years ago

Como posso saber da release com essa correção? Estou com o mesmo problema.

fredericocostachaves commented 6 years ago

https://github.com/caelum/vraptor-quartzjob/pull/16

2018-04-25 12:15 GMT-03:00 Anderson Scherrer notifications@github.com:

Como posso saber da release com essa correção? Estou com o mesmo problema.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/caelum/vraptor-quartzjob/issues/15#issuecomment-384324623, or mute the thread https://github.com/notifications/unsubscribe-auth/AL0gMXltbDKlGsQAJqw7CDKeQbGPlIyHks5tsJMvgaJpZM4MEXd- .

-- Atenciosamente, Frederico Costa Chaves