tangled.org / core
Monorepo for Tangled tangled.org
855
fork

Configure Feed

Select the types of activity you want to include in your feed.

appview,knotserver: validate git repo ownership according to knot #281

open opened by
oyster.cafe targeting master from lt/repo-rename-by-rkey
STACK 13
#350 appview,knotmirror: tolerate 1.13 knots during 1.14 rollout
0
#1
#349 appview,knotserver: 1.14 deploy prep
0
#2
#281 appview,knotserver: validate git repo ownership according to knot
0
#8
#276 knotmirror: key repos by repo_did + disk migration
0
#8
#275 knotmirror,spindle,appview: propagate repo rename across services
0
#7
#280 knotserver: rkey-based repo renaming
0
#8
#279 appview: handlers, state, and templates for more record types w/ repoDID
0
#8
#274 appview/oauth,notify: DID-based PDS record rewriting
0
#8
#278 appview: ingester and indexers use repoDID
0
#8
#273 appview: drop DbWrapper, move jetstream helpers onto *DB
0
#8
#277 appview/db: queries use repo_did lookups
0
#8
#272 appview/db,models: add repo_did columns and update model structs
0
#8
#271 lexicons,api: migrate stars+issues+pulls+collabs to repoDID
0
#8
Labels

None yet.

assignee

None yet.

Participants 1
AT URI
at://did:plc:3fwecdnvtcscjnrx2p4n7alz/sh.tangled.repo.pull/3mjwlqwlv7f22
+4 -4
Interdiff #7 → #8
api/tangled/repodescribeRepo.go

This file has not been changed.

+4 -4
appview/ingester.go
··· 1027 1027 return fmt.Errorf("failed to validate issue: %w", err) 1028 1028 } 1029 1029 1030 - if record.Repo != "" && !strings.HasPrefix(record.Repo, "did:") { 1031 - repo, repoErr := db.GetRepoByAtUri(i.Db, record.Repo) 1030 + if record.Repo != nil { 1031 + repo, repoErr := db.GetRepoByAtUri(i.Db, *record.Repo) 1032 1032 if repoErr == nil && repo.RepoDid != "" { 1033 1033 if enqErr := db.EnqueuePdsRecordMigration(ctx, i.Db, "add-repo-did", syntax.DID(did), syntax.NSID(tangled.RepoIssueNSID), syntax.RecordKey(e.Commit.RKey)); enqErr != nil { 1034 1034 l.Warn("failed to enqueue PDS rewrite for issue", "err", enqErr, "did", did, "repoDid", repo.RepoDid) ··· 1049 1049 return fmt.Errorf("failed to validate issue: %w", err) 1050 1050 } 1051 1051 1052 - if record.Repo != nil { 1053 - repo, repoErr := db.GetRepoByAtUri(i.Db, *record.Repo) 1052 + if record.Repo != "" && !strings.HasPrefix(record.Repo, "did:") { 1053 + repo, repoErr := db.GetRepoByAtUri(i.Db, record.Repo) 1054 1054 if repoErr == nil && repo.RepoDid != "" { 1055 1055 if enqErr := db.EnqueuePdsRecordMigration(ctx, i.Db, "add-repo-did", syntax.DID(did), syntax.NSID(tangled.RepoIssueNSID), syntax.RecordKey(e.Commit.RKey)); enqErr != nil { 1056 1056 l.Warn("failed to enqueue PDS rewrite for issue", "err", enqErr, "did", did, "repoDid", repo.RepoDid)
appview/ingester_repo.go

This file has not been changed.

appview/ingester_repo_test.go

This file has not been changed.

appview/pulls/comment.go

This file has not been changed.

appview/pulls/create.go

This patch was likely rebased, as context lines do not match.

appview/pulls/lifecycle.go

This file has not been changed.

appview/pulls/list.go

This file has not been changed.

appview/pulls/merge.go

This file has not been changed.

appview/pulls/resubmit.go

This file has not been changed.

appview/pulls/single.go

This patch was likely rebased, as context lines do not match.

appview/repoverify/verify.go

This file has not been changed.

appview/repoverify/verify_test.go

This file has not been changed.

appview/state/state.go

This file has not been changed.

knotserver/xrpc/repo_describe_repo.go

This file has not been changed.

knotserver/xrpc/repo_describe_repo_test.go

This file has not been changed.

knotserver/xrpc/xrpc.go

This file has not been changed.

lexicons/repo/describeRepo.json

This file has not been changed.

History

9 rounds 0 comments
sign up or login to add to the discussion
oyster.cafe submitted #8
diff
1 commit
expand
7895692c
appview,knotserver: validate git repo ownership according to knot
merge conflicts detected
expand
  • api/tangled/cbor_gen.go:866
  • api/tangled/feedstar.go:5
  • api/tangled/gitrefUpdate.go:29
  • api/tangled/repocollaborator.go:19
  • api/tangled/repoissue.go:22
  • api/tangled/repopull.go:39
  • api/tangled/tangledrepo.go:24
  • cmd/cborgen/cborgen.go:17
  • knotserver/xrpc/merge.go:118
  • lexicons/feed/star.json:10
  • lexicons/git/refUpdate.json:11
  • lexicons/issue/issue.json:9
  • lexicons/pulls/pull.json:65
  • lexicons/repo/collaborator.json:11
  • lexicons/repo/repo.json:6
expand 0 comments
oyster.cafe submitted #7
diff interdiff
1 commit
expand
b040abd5
appview,knotserver: validate git repo ownership according to knot
expand 0 comments
oyster.cafe submitted #6
diff interdiff
1 commit
expand
034da106
appview,knotserver: validate git repo ownership according to knot
expand 0 comments
oyster.cafe submitted #5
diff interdiff
1 commit
expand
034da106
appview,knotserver: validate git repo ownership according to knot
expand 0 comments
oyster.cafe submitted #4
diff interdiff
1 commit
expand
48a17df6
appview,knotserver: validate git repo ownership according to knot
expand 0 comments
oyster.cafe submitted #3
diff interdiff
1 commit
expand
7c8c9ee9
appview,knotserver: validate git repo ownership according to knot
expand 0 comments
oyster.cafe submitted #2
diff interdiff
1 commit
expand
653de4bf
appview,knotserver: validate git repo ownership according to knot
expand 0 comments
oyster.cafe submitted #1
diff interdiff
1 commit
expand
d24d0df8
appview,knotserver: validate git repo ownership according to knot
expand 0 comments
oyster.cafe submitted #0
diff
1 commit
expand
691a8d93
appview,knotserver: validate git repo ownership according to knot
expand 0 comments