From 8fe649cb5f12f990ae067d6c80293c0dd241731c Mon Sep 17 00:00:00 2001 From: Santo Cariotti Date: Thu, 18 Mar 2021 12:12:46 +0100 Subject: chore: add git error type and equals macro --- src/errors.rs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/errors.rs b/src/errors.rs index 8140cfe..1efb082 100644 --- a/src/errors.rs +++ b/src/errors.rs @@ -4,11 +4,12 @@ use serde::Serialize; use std::fmt; use tokio_postgres::error::Error; -#[derive(Debug)] +#[derive(Debug, Eq, PartialEq)] pub enum AppErrorType { DbError, NotFoundError, AuthorizationError, + GitError, } #[derive(Debug)] @@ -72,6 +73,7 @@ impl ResponseError for AppError { AppErrorType::DbError => StatusCode::INTERNAL_SERVER_ERROR, AppErrorType::NotFoundError => StatusCode::NOT_FOUND, AppErrorType::AuthorizationError => StatusCode::UNAUTHORIZED, + AppErrorType::GitError => StatusCode::BAD_REQUEST, } } -- cgit v1.2.3-18-g5258 From 23090f94334a662e11a3120fc84cf7a1e7e983b0 Mon Sep 17 00:00:00 2001 From: Santo Cariotti Date: Thu, 18 Mar 2021 12:13:31 +0100 Subject: feat(commit): create new commits from vec --- src/commit/models.rs | 46 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 46 insertions(+) (limited to 'src') diff --git a/src/commit/models.rs b/src/commit/models.rs index 8b81a15..2f1536b 100644 --- a/src/commit/models.rs +++ b/src/commit/models.rs @@ -93,4 +93,50 @@ impl Commit { }), } } + + /// Create commits from an array + pub async fn create( + pool: Pool, + commits: Vec, + ) -> Result, AppError> { + let client = get_client(pool.clone()).await.unwrap(); + let mut raw_query = "INSERT INTO commit VALUES".to_string(); + + for commit in commits { + let tree = match commit.tree { + Some(t) => format!("'{}'", t), + None => "NULL".to_string(), + }; + raw_query += &format!( + "('{}', {}, '{}', '{}', '{}', '{}', '{}', '{}', '{}'),", + commit.hash, + tree, + commit.text, + commit.date, + commit.author_email, + commit.author_name, + commit.committer_email, + commit.committer_name, + commit.repository_url + )[..] + } + + // Remove the last `,` + let _ = raw_query.pop(); + + // TODO: write query with &commits and parameter. Need to implement + // ToSql trait for `Commit` model + // let statement = client.prepare(&query[..]).await?; + // client.query(&statement, &[&commits] + + let statement = client.prepare(&raw_query[..]).await?; + let result = client + .query(&statement, &[]) + .await? + .iter() + .map(|row| Commit::from_row_ref(row).unwrap()) + .collect::>(); + + Ok(result) + } } -- cgit v1.2.3-18-g5258 From 4289e4961b6ba48a58e3719206d80b9cc9e795be Mon Sep 17 00:00:00 2001 From: Santo Cariotti Date: Thu, 18 Mar 2021 12:13:58 +0100 Subject: feat: add 'git' module --- src/git.rs | 142 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 142 insertions(+) create mode 100644 src/git.rs (limited to 'src') diff --git a/src/git.rs b/src/git.rs new file mode 100644 index 0000000..76f9a4e --- /dev/null +++ b/src/git.rs @@ -0,0 +1,142 @@ +use crate::commit::models::Commit; +use chrono::{DateTime, Local}; +use git2::{BranchType, Error, Repository, Time}; + +use std::fs::remove_dir_all; + +/// Return the temporary folder of the git repository +fn get_tmp_dir(repo_name: &String) -> String { + format!("/tmp/{}", &repo_name) +} + +/// Clone a repository `repo` in a temporary folder. It uses only GitHub at +/// this moment. +pub fn clone_repo(repo_name: &String) -> Result { + let url: &str = + &format!("https://github.com/{}", &repo_name).to_owned()[..]; + let tmp_dir: String = get_tmp_dir(&repo_name); + + let cloned = Repository::clone(url, tmp_dir)?; + + Ok(cloned) +} + +/// Check if a `branch` exists inside the repository and then set the head of +/// `repo` to that branch +pub fn get_branch(repo: &Repository, branch: &str) -> Result<(), Error> { + if let Err(e) = repo.find_branch(&branch, BranchType::Local) { + return Err(e); + } + + repo.set_head(&format!("refs/heads/{}", branch).to_owned()[..]) +} + +/// Get a `git2::Commit` and returns a date in RFC2822 format +pub fn get_date(commit: &git2::Commit) -> String { + let t: Time = commit.time(); + let (offset, sign) = match t.offset_minutes() { + n if n < 0 => (-n, '-'), + n => (n, '+'), + }; + let (hours, minutes) = (offset / 60, offset % 60); + let ts = + time::Timespec::new(t.seconds() + (t.offset_minutes() as i64) * 60, 0); + let time = time::at(ts); + + let result = format!( + "{} {}{:02}{:02}", + time.strftime("%a, %e %b %Y %T").unwrap(), + sign, + hours, + minutes + ); + + result +} + +/// Get a `git2::Commit` and returns a valid `Commit` to upload to the database +fn get_commit(gcommit: &git2::Commit, repo_name: &String) -> Commit { + let hash = gcommit.id().to_string(); + let tree = match gcommit.parent_id(0) { + Ok(parent) => Some(parent.to_string()), + Err(_) => None, + }; + + let mut text = "".to_string(); + for line in String::from_utf8_lossy(gcommit.message_bytes()).lines() { + text += line; + text += "\n"; + } + + // Remove the last "\n" + let _ = text.pop(); + + let date: DateTime = + match DateTime::parse_from_rfc2822(&get_date(&gcommit)) { + Ok(date) => date.into(), + Err(e) => { + // This is a real problem! + // TODO: manage this error + panic!(e) + } + }; + let author_email = gcommit.author().email().unwrap().to_string(); + let author_name = gcommit.author().name().unwrap().to_string(); + let committer_email = gcommit.committer().email().unwrap().to_string(); + let committer_name = gcommit.committer().name().unwrap().to_string(); + + Commit { + hash, + tree, + text, + date, + author_email, + author_name, + committer_email, + committer_name, + repository_url: repo_name.clone(), + } +} + +/// Get all commits from a Git repository. Returns an array of `Commit`. +/// First, clone the repo into a temporary folder. +/// Then, open the repository. +/// Then, get commits +/// Finally, remove the temporary folder +pub fn repo_commits(repo_name: &String) -> Result, Error> { + // Try to clone the repo. If it returns an error, it's useless to go ahead: + // raises an error. + let repo = match clone_repo(&repo_name) { + Ok(r) => r, + Err(e) => { + return Err(e); + } + }; + + if let Err(e) = get_branch(&repo, "main") { + return Err(e); + } + + let mut revwalk = repo.revwalk()?; + let head = repo.head().unwrap().target().unwrap(); + + if let Err(e) = revwalk.push(head) { + return Err(e); + } + + let mut commits: Vec = vec![]; + for commit in revwalk { + let hash = repo.find_commit(commit?)?; + commits.push(get_commit(&hash, &repo_name)); + } + + if let Err(_) = remove_dir_all(get_tmp_dir(&repo_name)) { + return Err(git2::Error::new( + git2::ErrorCode::GenericError, + git2::ErrorClass::Os, + "Temporary clone not deleted", + )); + } + + Ok(commits) +} -- cgit v1.2.3-18-g5258 From 5f4f01791d26003889afbb7179633ca98d5647e1 Mon Sep 17 00:00:00 2001 From: Santo Cariotti Date: Thu, 18 Mar 2021 12:14:15 +0100 Subject: feat: 'git' module imported --- src/main.rs | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src') diff --git a/src/main.rs b/src/main.rs index 6f20201..2e499f9 100644 --- a/src/main.rs +++ b/src/main.rs @@ -3,6 +3,8 @@ mod db; mod errors; mod helpers; +mod git; + mod branch; mod commit; mod email; -- cgit v1.2.3-18-g5258 From 214137b65413daff684cfe8df4d544ea0cb78516 Mon Sep 17 00:00:00 2001 From: Santo Cariotti Date: Thu, 18 Mar 2021 12:14:32 +0100 Subject: feat: add Commit(s) when repository is created --- src/repository/models.rs | 46 +++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 45 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/repository/models.rs b/src/repository/models.rs index e3d8456..5b7b445 100644 --- a/src/repository/models.rs +++ b/src/repository/models.rs @@ -1,5 +1,8 @@ +use crate::commit::models::Commit; use crate::db::get_client; +use crate::email::models::{Email, EmailData}; use crate::errors::{AppError, AppErrorType}; +use crate::git; use crate::helpers::name_of_git_repository; use chrono::NaiveDateTime; @@ -9,6 +12,7 @@ use tokio_pg_mapper::FromTokioPostgresRow; use tokio_pg_mapper_derive::PostgresMapper; use uuid::Uuid; +use std::collections::HashSet; use std::net::SocketAddr; #[derive(Serialize, Deserialize, PostgresMapper)] @@ -185,7 +189,47 @@ impl Repository { .map(|row| Repository::from_row_ref(&row).unwrap()); match repo { - Some(repo) => Ok(repo), + Some(repo) => { + let commits = match git::repo_commits(&repo_name) { + Ok(c) => c, + Err(e) => { + return Err(AppError { + message: Some( + format!( + "Repository couldn't be created now: {:?}", + e + ) + .to_string(), + ), + cause: Some("Repository clone".to_string()), + error_type: AppErrorType::GitError, + }) + } + }; + + let mut emails: HashSet = HashSet::new(); + for commit in &commits { + emails.insert(commit.author_email.clone()); + emails.insert(commit.committer_email.clone()); + } + for email in emails { + if let Err(e) = + Email::create(pool.clone(), &EmailData { email }).await + { + if e.error_type == AppErrorType::DbError { + return Err(e); + } + } + } + + let commits_result = + Commit::create(pool.clone(), commits).await; + if let Err(e) = commits_result { + panic!("{}", e); + } + + Ok(repo) + } None => Err(AppError { message: Some("Error creating a new repository".to_string()), cause: Some("Unknown error".to_string()), -- cgit v1.2.3-18-g5258 From 7bdd6d8f2d18022c34986a6eea4620d8eb6dcb3a Mon Sep 17 00:00:00 2001 From: Santo Cariotti Date: Thu, 18 Mar 2021 12:18:19 +0100 Subject: fix: handle error for commit creation on repos --- src/repository/models.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/repository/models.rs b/src/repository/models.rs index 5b7b445..de82d45 100644 --- a/src/repository/models.rs +++ b/src/repository/models.rs @@ -225,7 +225,7 @@ impl Repository { let commits_result = Commit::create(pool.clone(), commits).await; if let Err(e) = commits_result { - panic!("{}", e); + return Err(e); } Ok(repo) -- cgit v1.2.3-18-g5258