agola icon indicating copy to clipboard operation
agola copied to clipboard

vendor: update gitea sdk v0.12.0 -> 0.14.0

Open 6543 opened this issue 5 years ago • 9 comments

and adopt changes

6543 avatar Sep 16 '20 07:09 6543

I think main issue is fixed with v0.13.1 ...

-> https://gitea.com/gitea/go-sdk/pulls/429

6543 avatar Sep 29 '20 15:09 6543

@sgotti the gitea version agola test against is too old, it just turns out the new version do not run properly?

Can you give me a hint where the gitea test instance is setup for the test?

6543 avatar Apr 08 '21 11:04 6543

@sgotti this pull is finally ready :rocket:

6543 avatar Apr 09 '21 08:04 6543

@6543 Thanks for the PR.

I noticed that now the call to gitea NewClient can return an error since it calls the server to check the minimal version. This is quite different from other client implementations where they just create a client without doing any call so you can cache and use the client when needed. In such case if in the moment of the call to NewClient the server is temporarily unreachable the call will fail).

Anyway, does this mean that the minimal required gitea version is 1.11?

sgotti avatar Apr 13 '21 15:04 sgotti

@sgotti yes gitea >= 1.11 requirement

6543 avatar Apr 13 '21 15:04 6543

@sgotti what's up? do you need more information?

6543 avatar Apr 27 '21 15:04 6543

@6543 No, just not enough time. My only worries were related to the client checking the Gitea version and I wanted to check if Gitea 1.11 is really needed. The current client as far as I know works with all the versions of Gitea, so also the new one should work also with version before 1.11.

sgotti avatar Apr 27 '21 15:04 sgotti

by version checking we avoid unwanted & unpredictable behaviour - since api got a lot of changes since then. beside that and the fact that v1.12 start pagination support, you really should not run gitea below v1.13.7 for security reasons ...

@sgotti so in my opinion if one run a gitea older than v1.11 he either also will likely stick with an older agola version or do custom patches anyway ...

so I'm not sure why agola should run with all gitea versions, witch it in fact never did since early versions do miss a lot of api features you use ...


PS: yes take your time to test this patch

6543 avatar Apr 27 '21 18:04 6543

@sgotti resolved conflict and adjust to current codebase

6543 avatar Aug 30 '22 19:08 6543

Already updated in #385

sgotti avatar Jul 24 '23 10:07 sgotti