Suggested change to PreRepositoryHook

[Tomas Bjerre]

My plugin is reporting findings from the commit messages when commits are pushed. It does not necessarily have to be findings that should reject the commit from being pushed. It can be information, or warnings, that should just be communicated.

https://github.com/tomasbjerre/simple-bitbucket-commit-checker/blob/master/src/main/java/se/bjurr/sbcc/SbccPreReceiveRepositoryHook.java#L98

If I understand correctly I should, from BBS 6.0, implement this class instead:

https://developer.atlassian.com/static/javadoc/bitbucket-server/latest/spi/reference/com/atlassian/bitbucket/hook/repository/PreRepositoryHook.html#preUpdate(com.atlassian.bitbucket.hook.repository.PreRepositoryHookContext

If I understand correctly, it can only report feedback with RepositoryHookResult and only when commits are rejected. It cannot report back a text message regarding valid accepted commits.

I would suggest that being changed. That was pretty much the only reason that I started working on this plugin.

A quick googling did not tell my how to make such a request!

[Julius Davies [bit-booster.com] · Accepted solution]

I haven't ported my own hook to the new BB 6 approach, but from clicking around the javadocs a little, I think you can write messages using the 2nd parameter "T request" of the preUpdate() call.    This "request" param will probably implement RepositoryHookRequest, in which case you can call request.getScmHookDetails().out().