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

Fixa Repository#clone_url #152

Closed
jesjos opened this issue May 2, 2012 · 5 comments
Closed

Fixa Repository#clone_url #152

jesjos opened this issue May 2, 2012 · 5 comments
Assignees
Labels

Comments

@jesjos
Copy link
Member

jesjos commented May 2, 2012

Denna metod är just nu en gammal gitoriouskvarleva som försöker använda repo_owner och lite annat bös som inte finns kvar.
Vet @oleander eller @Tarrasch hur en sådan url skulle kunna se ut?

@oleander
Copy link
Member

oleander commented May 2, 2012

De finns ingen möjlighet att från en given repository lista ut vad containern tillhör och på så sätt generera en korrelt url.

Detta då en Repository i dagsläget kan tillhöra:

  • LabHasGroup
  • InitialLabCommit
  • Hook (kan vara utdaterad)
  • Committership (kan vara utdaterad)
  • Cloner (kan vara utdaterad)

Detta gäller dock inte om du bara vill skapa en "ful" länk till repon.
Vi kan alltid implementera stöd för http://w.com/repositories/123.git
Men de vore snyggare att bara tillåta exempelvis http://w.com/courses/1/labs/1.git.
Nyss nämnda url går bara att generera om vi har en repository och en förälder, i de här fallet en LabHasGroup.

@Tarrasch
Copy link
Contributor

Tarrasch commented May 2, 2012

Som oleander sa, fast angående en "ful" url så kan vi alltid skapa en http://w.com/dea/dbe/ef1234568abcabcabcabc.git, det skulle vara mycket enklare än /repositories/123.git, då kan man heller inte accessa sig till någon random repo genom att gissa en repo id.

@jesjos, skulle det vara ok om du istället kunde få LabHasGroup#student_url? En ide är att låta urlar som http://w.com/dea/dbe/ef1234568abcabcabcabc.git vara read-only urlar för assistenter och så, den skulle kunna ligga i Repository, eller, det är nog den som redan finns.

@jesjos
Copy link
Member Author

jesjos commented May 2, 2012

Okej. Vi behöver två sorters url:ar

  1. En read-write-url för studenter
  2. En read-only-url för assistenter och kanske examinator.

@Tarrasch och jag diskuterade det IRL och föreslår:

  1. I LabHasGroup har vi en student_url eller liknande som ger en read-write-url, som kan vara snygg
  2. I Repository har vi clone_url som ger en "ful" read-only-url i stilen http://w.com/dea/dbe/ef1234568abcabcabcabc.git

@Tarrasch
Copy link
Contributor

Tarrasch commented May 2, 2012

Alternativt kan handledare och studenter ha samma url enligt mitt förslag i #76, jag kör på det om inte Linus protesterar

@Tarrasch
Copy link
Contributor

Tarrasch commented May 6, 2012

Heads up bara, ni måste ändra i er config-mapp enligt commit b9f125b för att clone-urin ska fungera.

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

No branches or pull requests

3 participants