From 88110f536f9c3843ecf5016122e101f8a424af77 Mon Sep 17 00:00:00 2001 From: ivarlovlie Date: Sat, 22 Jan 2022 22:43:38 +0100 Subject: Initial commit --- .../Api/Internal/Account/CreateSessionRequest.cs | 8 +++ .../Api/Internal/Account/CreateSessionRoute.cs | 41 ++++++++++++ .../Api/Internal/Account/CreateTokenRequest.cs | 6 ++ .../Api/Internal/Account/CreateTokenRoute.cs | 34 ++++++++++ .../Api/Internal/Account/DeleteTokenRoute.cs | 24 +++++++ src/server/Api/Internal/Account/EndSessionRoute.cs | 13 ++++ src/server/Api/Internal/Account/GetArchiveRoute.cs | 36 +++++++++++ .../Api/Internal/Account/GetProfileDataRoute.cs | 11 ++++ src/server/Api/Internal/Account/GetTokensRoute.cs | 17 +++++ .../Api/Internal/Account/UpdatePasswordRequest.cs | 6 ++ .../Api/Internal/Account/UpdatePasswordRoute.cs | 35 +++++++++++ src/server/Api/Internal/BaseInternalRoute.cs | 15 +++++ src/server/Api/Internal/Dtos/SiteReportDto.cs | 8 +++ src/server/Api/Internal/Dtos/UserArchiveDto.cs | 32 ++++++++++ src/server/Api/Internal/GetSiteReportRequest.cs | 6 ++ src/server/Api/Internal/GetSiteReportRoute.cs | 54 ++++++++++++++++ src/server/Api/Internal/LoggedInInternalUser.cs | 7 +++ src/server/Api/Internal/RouteBaseAsync.cs | 73 ++++++++++++++++++++++ src/server/Api/Internal/RouteBaseSync.cs | 53 ++++++++++++++++ src/server/Api/V1/ApiSpecV1.cs | 18 ++++++ src/server/Api/V1/BaseV1Route.cs | 17 +++++ src/server/Api/V1/Entries/CreateEntryRequest.cs | 27 ++++++++ src/server/Api/V1/Entries/CreateEntryRoute.cs | 34 ++++++++++ src/server/Api/V1/Entries/DeleteEntryRoute.cs | 30 +++++++++ src/server/Api/V1/Entries/Dtos/EntryDto.cs | 16 +++++ src/server/Api/V1/Entries/GetEntriesRoute.cs | 21 +++++++ src/server/Api/V1/Entries/UpdateEntryRequest.cs | 18 ++++++ src/server/Api/V1/Entries/UpdateEntryRoute.cs | 45 +++++++++++++ src/server/Api/V1/LoggedInV1User.cs | 7 +++ src/server/Api/V1/RouteBaseV1Async.cs | 73 ++++++++++++++++++++++ src/server/Api/V1/RouteBaseV1Sync.cs | 53 ++++++++++++++++ 31 files changed, 838 insertions(+) create mode 100644 src/server/Api/Internal/Account/CreateSessionRequest.cs create mode 100644 src/server/Api/Internal/Account/CreateSessionRoute.cs create mode 100644 src/server/Api/Internal/Account/CreateTokenRequest.cs create mode 100644 src/server/Api/Internal/Account/CreateTokenRoute.cs create mode 100644 src/server/Api/Internal/Account/DeleteTokenRoute.cs create mode 100644 src/server/Api/Internal/Account/EndSessionRoute.cs create mode 100644 src/server/Api/Internal/Account/GetArchiveRoute.cs create mode 100644 src/server/Api/Internal/Account/GetProfileDataRoute.cs create mode 100644 src/server/Api/Internal/Account/GetTokensRoute.cs create mode 100644 src/server/Api/Internal/Account/UpdatePasswordRequest.cs create mode 100644 src/server/Api/Internal/Account/UpdatePasswordRoute.cs create mode 100644 src/server/Api/Internal/BaseInternalRoute.cs create mode 100644 src/server/Api/Internal/Dtos/SiteReportDto.cs create mode 100644 src/server/Api/Internal/Dtos/UserArchiveDto.cs create mode 100644 src/server/Api/Internal/GetSiteReportRequest.cs create mode 100644 src/server/Api/Internal/GetSiteReportRoute.cs create mode 100644 src/server/Api/Internal/LoggedInInternalUser.cs create mode 100644 src/server/Api/Internal/RouteBaseAsync.cs create mode 100644 src/server/Api/Internal/RouteBaseSync.cs create mode 100644 src/server/Api/V1/ApiSpecV1.cs create mode 100644 src/server/Api/V1/BaseV1Route.cs create mode 100644 src/server/Api/V1/Entries/CreateEntryRequest.cs create mode 100644 src/server/Api/V1/Entries/CreateEntryRoute.cs create mode 100644 src/server/Api/V1/Entries/DeleteEntryRoute.cs create mode 100644 src/server/Api/V1/Entries/Dtos/EntryDto.cs create mode 100644 src/server/Api/V1/Entries/GetEntriesRoute.cs create mode 100644 src/server/Api/V1/Entries/UpdateEntryRequest.cs create mode 100644 src/server/Api/V1/Entries/UpdateEntryRoute.cs create mode 100644 src/server/Api/V1/LoggedInV1User.cs create mode 100644 src/server/Api/V1/RouteBaseV1Async.cs create mode 100644 src/server/Api/V1/RouteBaseV1Sync.cs (limited to 'src/server/Api') diff --git a/src/server/Api/Internal/Account/CreateSessionRequest.cs b/src/server/Api/Internal/Account/CreateSessionRequest.cs new file mode 100644 index 0000000..24c28b4 --- /dev/null +++ b/src/server/Api/Internal/Account/CreateSessionRequest.cs @@ -0,0 +1,8 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class CreateSessionRequest +{ + public string Username { get; set; } + public string Password { get; set; } + public bool Persist { get; set; } +} diff --git a/src/server/Api/Internal/Account/CreateSessionRoute.cs b/src/server/Api/Internal/Account/CreateSessionRoute.cs new file mode 100644 index 0000000..09e05b6 --- /dev/null +++ b/src/server/Api/Internal/Account/CreateSessionRoute.cs @@ -0,0 +1,41 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class CreateSessionRoute : RouteBaseInternalAsync.WithRequest.WithActionResult +{ + private readonly AppDbContext _context; + + public CreateSessionRoute(AppDbContext context) { + _context = context; + } + + [AllowAnonymous] + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpPost("~/v{version:apiVersion}/account/create-session")] + public override async Task HandleAsync(CreateSessionRequest payload, CancellationToken cancellationToken = default) { + var user = _context.Users.SingleOrDefault(u => u.Username == payload.Username); + if (user == default || !user.VerifyPassword(payload.Password)) { + return BadRequest(new ErrorResult("Invalid username or password")); + } + + var claims = user.DefaultClaims(); + var identity = new ClaimsIdentity(claims, CookieAuthenticationDefaults.AuthenticationScheme); + var principal = new ClaimsPrincipal(identity); + var authenticationProperties = new AuthenticationProperties { + AllowRefresh = true, + IssuedUtc = DateTimeOffset.UtcNow, + }; + + if (payload.Persist) { + authenticationProperties.ExpiresUtc = DateTimeOffset.UtcNow.AddYears(100); + authenticationProperties.IsPersistent = true; + } + + await HttpContext.SignInAsync(principal, authenticationProperties); + // Return new LoggedInInternalUser here, because it is not materialised in AppControllerBase yet. + return Ok(new LoggedInInternalUser { + Id = user.Id, + Username = user.Username + }); + } +} diff --git a/src/server/Api/Internal/Account/CreateTokenRequest.cs b/src/server/Api/Internal/Account/CreateTokenRequest.cs new file mode 100644 index 0000000..399bdfc --- /dev/null +++ b/src/server/Api/Internal/Account/CreateTokenRequest.cs @@ -0,0 +1,6 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class CreateTokenRequest +{ + public string Name { get; set; } +} diff --git a/src/server/Api/Internal/Account/CreateTokenRoute.cs b/src/server/Api/Internal/Account/CreateTokenRoute.cs new file mode 100644 index 0000000..ea0e01f --- /dev/null +++ b/src/server/Api/Internal/Account/CreateTokenRoute.cs @@ -0,0 +1,34 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class CreateTokenRoute : RouteBaseInternalSync.WithRequest.WithActionResult +{ + private readonly AppDbContext _context; + + public CreateTokenRoute(AppDbContext context) { + _context = context; + } + + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpPost("~/v{version:apiVersion}/account/create-token")] + public override ActionResult Handle(CreateTokenRequest request) { + var user = _context.Users.SingleOrDefault(c => c.Id == LoggedInUser.Id); + if (user == default) { + return NotFound(new ErrorResult("User does not exist")); + } + + if (request.Name.IsNullOrWhiteSpace()) { + return BadRequest(new ErrorResult("Token name is required")); + } + + var token = new AccessToken { + Id = Guid.NewGuid(), + Name = request.Name, + User = user + }; + + _context.AccessTokens.Add(token); + _context.SaveChanges(); + return Ok(token); + } +} diff --git a/src/server/Api/Internal/Account/DeleteTokenRoute.cs b/src/server/Api/Internal/Account/DeleteTokenRoute.cs new file mode 100644 index 0000000..f423b6f --- /dev/null +++ b/src/server/Api/Internal/Account/DeleteTokenRoute.cs @@ -0,0 +1,24 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class DeleteTokenRoute : RouteBaseInternalSync.WithRequest.WithActionResult +{ + private readonly AppDbContext _context; + + public DeleteTokenRoute(AppDbContext context) { + _context = context; + } + + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpDelete("~/v{version:apiVersion}/account/delete-token")] + public override ActionResult Handle(Guid id) { + var token = _context.AccessTokens.SingleOrDefault(c => c.Id == id); + if (token == default) { + return NotFound(); + } + + _context.AccessTokens.Remove(token); + _context.SaveChanges(); + return Ok(); + } +} diff --git a/src/server/Api/Internal/Account/EndSessionRoute.cs b/src/server/Api/Internal/Account/EndSessionRoute.cs new file mode 100644 index 0000000..4f32168 --- /dev/null +++ b/src/server/Api/Internal/Account/EndSessionRoute.cs @@ -0,0 +1,13 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class EndSessionRoute : RouteBaseInternalAsync.WithoutRequest.WithActionResult +{ + [AllowAnonymous] + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpGet("~/v{version:apiVersion}/account/end-session")] + public override async Task HandleAsync(CancellationToken cancellationToken = default) { + await HttpContext.SignOutAsync(); + return Ok(); + } +} diff --git a/src/server/Api/Internal/Account/GetArchiveRoute.cs b/src/server/Api/Internal/Account/GetArchiveRoute.cs new file mode 100644 index 0000000..5dc006e --- /dev/null +++ b/src/server/Api/Internal/Account/GetArchiveRoute.cs @@ -0,0 +1,36 @@ +using IOL.BookmarkThing.Server.Api.Internal.Dtos; + +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class GetArchiveRoute : RouteBaseInternalSync.WithoutRequest.WithActionResult +{ + private readonly AppDbContext _context; + + public GetArchiveRoute(AppDbContext context) { + _context = context; + } + + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpGet("~/v{version:apiVersion}/account/archive")] + public override ActionResult Handle() { + var user = _context.Users.SingleOrDefault(c => c.Id == LoggedInUser.Id); + if (user == default) { + return NotFound(); + } + + var entries = _context.Entries.Where(c => c.UserId == user.Id); + var archive = new UserArchiveDto() { + Created = DateTime.UtcNow, + User = new UserArchiveDto.UserArchiveUser(user), + Entries = entries.Select(c => new UserArchiveDto.UserArchiveEntry(c)).ToList() + }; + var jsonOptions = new JsonSerializerOptions { + WriteIndented = true + }; + var archiveBytes = JsonSerializer.SerializeToUtf8Bytes(archive, jsonOptions); + return File(archiveBytes, + "application/json", + "bookmark-thing-archive-" + user.Username + "-" + DateTime.UtcNow.ToString("yyyyMMddTHHmmss") + ".json"); + } +} diff --git a/src/server/Api/Internal/Account/GetProfileDataRoute.cs b/src/server/Api/Internal/Account/GetProfileDataRoute.cs new file mode 100644 index 0000000..adf1cba --- /dev/null +++ b/src/server/Api/Internal/Account/GetProfileDataRoute.cs @@ -0,0 +1,11 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class GetProfileDataRoute : RouteBaseInternalSync.WithoutRequest.WithActionResult +{ + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpGet("~/v{version:apiVersion}/account/profile-data")] + public override ActionResult Handle() { + return Ok(LoggedInUser); + } +} diff --git a/src/server/Api/Internal/Account/GetTokensRoute.cs b/src/server/Api/Internal/Account/GetTokensRoute.cs new file mode 100644 index 0000000..7e87bc7 --- /dev/null +++ b/src/server/Api/Internal/Account/GetTokensRoute.cs @@ -0,0 +1,17 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class GetTokensRoute : RouteBaseInternalSync.WithoutRequest.WithResult>> +{ + private readonly AppDbContext _context; + + public GetTokensRoute(AppDbContext context) { + _context = context; + } + + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpGet("~/v{version:apiVersion}/account/tokens")] + public override ActionResult> Handle() { + return Ok(_context.AccessTokens.Where(c => c.User.Id == LoggedInUser.Id)); + } +} diff --git a/src/server/Api/Internal/Account/UpdatePasswordRequest.cs b/src/server/Api/Internal/Account/UpdatePasswordRequest.cs new file mode 100644 index 0000000..1773bb0 --- /dev/null +++ b/src/server/Api/Internal/Account/UpdatePasswordRequest.cs @@ -0,0 +1,6 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class UpdatePasswordRequest +{ + public string NewPassword { get; set; } +} diff --git a/src/server/Api/Internal/Account/UpdatePasswordRoute.cs b/src/server/Api/Internal/Account/UpdatePasswordRoute.cs new file mode 100644 index 0000000..d06e850 --- /dev/null +++ b/src/server/Api/Internal/Account/UpdatePasswordRoute.cs @@ -0,0 +1,35 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Account; + +public class UpdatePasswordRoute : RouteBaseInternalSync.WithRequest.WithActionResult +{ + private readonly AppDbContext _context; + + public UpdatePasswordRoute(AppDbContext context) { + _context = context; + } + + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpPost("~/v{version:apiVersion}/account/update-password")] + public override ActionResult Handle(UpdatePasswordRequest payload) { + if (payload.NewPassword.IsNullOrWhiteSpace()) { + return BadRequest(new ErrorResult("Invalid request", + "The new password field is required")); + } + + if (payload.NewPassword.Length < 6) { + return BadRequest(new ErrorResult("Invalid request", + "The new password must contain atleast 6 characters")); + } + + var user = _context.Users.SingleOrDefault(c => c.Id == LoggedInUser.Id); + if (user == default) { + HttpContext.SignOutAsync(); + return StatusCode(403); + } + + user.HashAndSetPassword(payload.NewPassword); + _context.SaveChanges(); + return Ok(); + } +} diff --git a/src/server/Api/Internal/BaseInternalRoute.cs b/src/server/Api/Internal/BaseInternalRoute.cs new file mode 100644 index 0000000..2f92c8e --- /dev/null +++ b/src/server/Api/Internal/BaseInternalRoute.cs @@ -0,0 +1,15 @@ +namespace IOL.BookmarkThing.Server.Api.Internal; + +/// +[Authorize] +[ApiController] +public class BaseInternalRoute : ControllerBase +{ + /// + /// User data for the currently logged on user. + /// + protected LoggedInInternalUser LoggedInUser => new() { + Username = User.Identity?.Name, + Id = User.Claims.SingleOrDefault(c => c.Type == ClaimTypes.NameIdentifier)?.Value.ToGuid() ?? default + }; +} diff --git a/src/server/Api/Internal/Dtos/SiteReportDto.cs b/src/server/Api/Internal/Dtos/SiteReportDto.cs new file mode 100644 index 0000000..9c4aba8 --- /dev/null +++ b/src/server/Api/Internal/Dtos/SiteReportDto.cs @@ -0,0 +1,8 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Dtos; + +public class SiteReportDto +{ + public string Description { get; set; } + public bool Duplicate { get; set; } + public bool Unreachable { get; set; } +} diff --git a/src/server/Api/Internal/Dtos/UserArchiveDto.cs b/src/server/Api/Internal/Dtos/UserArchiveDto.cs new file mode 100644 index 0000000..ec691b3 --- /dev/null +++ b/src/server/Api/Internal/Dtos/UserArchiveDto.cs @@ -0,0 +1,32 @@ +namespace IOL.BookmarkThing.Server.Api.Internal.Dtos; + +public class UserArchiveDto +{ + public DateTime Created { get; set; } + public UserArchiveUser User { get; set; } + public List Entries { get; set; } + + public class UserArchiveUser + { + public UserArchiveUser(User user) { + Created = user.Created; + Username = user.Username; + } + + public DateTime Created { get; set; } + public string Username { get; set; } + } + + public class UserArchiveEntry + { + public UserArchiveEntry(Entry entry) { + Url = entry.Url; + Description = entry.Description; + Tags = entry.Tags; + } + + public Uri Url { get; set; } + public string Description { get; set; } + public string Tags { get; set; } + } +} diff --git a/src/server/Api/Internal/GetSiteReportRequest.cs b/src/server/Api/Internal/GetSiteReportRequest.cs new file mode 100644 index 0000000..52fbfe8 --- /dev/null +++ b/src/server/Api/Internal/GetSiteReportRequest.cs @@ -0,0 +1,6 @@ +namespace IOL.BookmarkThing.Server.Api.Internal; + +public class GetSiteReportRequest +{ + public Uri Url { get; set; } +} diff --git a/src/server/Api/Internal/GetSiteReportRoute.cs b/src/server/Api/Internal/GetSiteReportRoute.cs new file mode 100644 index 0000000..58d6637 --- /dev/null +++ b/src/server/Api/Internal/GetSiteReportRoute.cs @@ -0,0 +1,54 @@ +using HtmlAgilityPack; +using IOL.BookmarkThing.Server.Api.Internal.Dtos; + +namespace IOL.BookmarkThing.Server.Api.Internal; + +public class GetSiteReportRoute : RouteBaseInternalAsync.WithRequest.WithActionResult +{ + private readonly HttpClient _client; + private readonly ILogger _logger; + private readonly AppDbContext _context; + + public GetSiteReportRoute(HttpClient client, ILogger logger, AppDbContext context) { + _client = client; + _logger = logger; + _context = context; + } + + [ApiVersionNeutral] + [ApiExplorerSettings(IgnoreApi = true)] + [HttpPost("~/v{version:apiVersion}/site-report")] + public override async Task HandleAsync(GetSiteReportRequest request, CancellationToken cancellationToken = default) { + var report = new SiteReportDto(); + var exists = _context.Entries.Any(c => c.Url == request.Url && c.UserId == LoggedInUser.Id); + if (exists) { + report.Duplicate = true; + return Ok(report); + } + + try { + var http_request = await _client.GetAsync(request.Url, cancellationToken); + if (http_request.IsSuccessStatusCode) { + try { + var document = new HtmlDocument(); + document.Load(await http_request.Content.ReadAsStreamAsync(cancellationToken)); + var htmlNodes = document.DocumentNode.Descendants("meta") + .Where(p => p.GetAttributeValue("name", string.Empty).Equals("description", StringComparison.InvariantCultureIgnoreCase)); + report.Description = htmlNodes.FirstOrDefault()?.GetAttributeValue("content", string.Empty); + } catch (Exception e) { + _logger.LogWarning(e, "An error occured when parsing site for site report"); + } + } else { + report.Unreachable = true; + } + } catch (Exception e) { + if (e is HttpRequestException) { + report.Unreachable = true; + } + + _logger.LogError(e, "An error occured when getting external site for site report"); + } + + return Ok(report); + } +} diff --git a/src/server/Api/Internal/LoggedInInternalUser.cs b/src/server/Api/Internal/LoggedInInternalUser.cs new file mode 100644 index 0000000..e08dd51 --- /dev/null +++ b/src/server/Api/Internal/LoggedInInternalUser.cs @@ -0,0 +1,7 @@ +namespace IOL.BookmarkThing.Server.Api.Internal; + +public class LoggedInInternalUser +{ + public Guid Id { get; set; } + public string Username { get; set; } +} diff --git a/src/server/Api/Internal/RouteBaseAsync.cs b/src/server/Api/Internal/RouteBaseAsync.cs new file mode 100644 index 0000000..b5ad709 --- /dev/null +++ b/src/server/Api/Internal/RouteBaseAsync.cs @@ -0,0 +1,73 @@ +namespace IOL.BookmarkThing.Server.Api.Internal; + +/// +/// A base class for an endpoint that accepts parameters. +/// +public static class RouteBaseInternalAsync +{ + public static class WithRequest + { + public abstract class WithResult : BaseInternalRoute + { + public abstract Task HandleAsync( + TRequest request, + CancellationToken cancellationToken = default + ); + } + + public abstract class WithoutResult : BaseInternalRoute + { + public abstract Task HandleAsync( + TRequest request, + CancellationToken cancellationToken = default + ); + } + + public abstract class WithActionResult : BaseInternalRoute + { + public abstract Task> HandleAsync( + TRequest request, + CancellationToken cancellationToken = default + ); + } + + public abstract class WithActionResult : BaseInternalRoute + { + public abstract Task HandleAsync( + TRequest request, + CancellationToken cancellationToken = default + ); + } + } + + public static class WithoutRequest + { + public abstract class WithResult : BaseInternalRoute + { + public abstract Task HandleAsync( + CancellationToken cancellationToken = default + ); + } + + public abstract class WithoutResult : BaseInternalRoute + { + public abstract Task HandleAsync( + CancellationToken cancellationToken = default + ); + } + + public abstract class WithActionResult : BaseInternalRoute + { + public abstract Task> HandleAsync( + CancellationToken cancellationToken = default + ); + } + + public abstract class WithActionResult : BaseInternalRoute + { + public abstract Task HandleAsync( + CancellationToken cancellationToken = default + ); + } + } +} diff --git a/src/server/Api/Internal/RouteBaseSync.cs b/src/server/Api/Internal/RouteBaseSync.cs new file mode 100644 index 0000000..128a3a9 --- /dev/null +++ b/src/server/Api/Internal/RouteBaseSync.cs @@ -0,0 +1,53 @@ +namespace IOL.BookmarkThing.Server.Api.Internal; + +/// +/// A base class for an endpoint that accepts parameters. +/// +public static class RouteBaseInternalSync +{ + public static class WithRequest + { + public abstract class WithResult : BaseInternalRoute + { + public abstract TResponse Handle(TRequest request); + } + + public abstract class WithoutResult : BaseInternalRoute + { + public abstract void Handle(TRequest request); + } + + public abstract class WithActionResult : BaseInternalRoute + { + public abstract ActionResult Handle(TRequest request); + } + + public abstract class WithActionResult : BaseInternalRoute + { + public abstract ActionResult Handle(TRequest entry); + } + } + + public static class WithoutRequest + { + public abstract class WithResult : BaseInternalRoute + { + public abstract TResponse Handle(); + } + + public abstract class WithoutResult : BaseInternalRoute + { + public abstract void Handle(); + } + + public abstract class WithActionResult : BaseInternalRoute + { + public abstract ActionResult Handle(); + } + + public abstract class WithActionResult : BaseInternalRoute + { + public abstract ActionResult Handle(); + } + } +} diff --git a/src/server/Api/V1/ApiSpecV1.cs b/src/server/Api/V1/ApiSpecV1.cs new file mode 100644 index 0000000..c1d1cbf --- /dev/null +++ b/src/server/Api/V1/ApiSpecV1.cs @@ -0,0 +1,18 @@ +namespace IOL.BookmarkThing.Server.Api.V1; + +public static class ApiSpecV1 +{ + private const int MAJOR = 1; + private const int MINOR = 0; + public const string VERSION_STRING = "1.0"; + + public static ApiSpecDocument Document => new() { + Version = new ApiVersion(MAJOR, MINOR), + VersionName = VERSION_STRING, + SwaggerPath = $"/swagger/{VERSION_STRING}/swagger.json", + OpenApiInfo = new OpenApiInfo { + Title = Constants.API_NAME, + Version = VERSION_STRING + } + }; +} diff --git a/src/server/Api/V1/BaseV1Route.cs b/src/server/Api/V1/BaseV1Route.cs new file mode 100644 index 0000000..ba7d978 --- /dev/null +++ b/src/server/Api/V1/BaseV1Route.cs @@ -0,0 +1,17 @@ +namespace IOL.BookmarkThing.Server.Api.V1; + +/// +[Authorize(AuthenticationSchemes = AuthSchemes)] +[ApiController] +public class BaseV1Route : ControllerBase +{ + private const string AuthSchemes = CookieAuthenticationDefaults.AuthenticationScheme + "," + Constants.BASIC_AUTH_SCHEME; + + /// + /// User data for the currently logged on user. + /// + protected LoggedInV1User LoggedInUser => new() { + Username = User.Identity?.Name, + Id = User.Claims.SingleOrDefault(c => c.Type == ClaimTypes.NameIdentifier)?.Value.ToGuid() ?? default + }; +} diff --git a/src/server/Api/V1/Entries/CreateEntryRequest.cs b/src/server/Api/V1/Entries/CreateEntryRequest.cs new file mode 100644 index 0000000..a9c5070 --- /dev/null +++ b/src/server/Api/V1/Entries/CreateEntryRequest.cs @@ -0,0 +1,27 @@ +namespace IOL.BookmarkThing.Server.Api.V1.Entries; + +public class CreateEntryRequest +{ + public Uri Url { get; set; } + public string Description { get; set; } + public string Tags { get; set; } + + public List GetErrors() { + var errors = new List(); + if (Url == default) { + errors.Add(new ErrorResult("Url is a required field")); + } + + return errors; + } + + public Entry AsDbEntity(Guid userId) { + return new Entry { + Id = Guid.NewGuid(), + UserId = userId, + Url = Url, + Description = Description, + Tags = Tags + }; + } +} diff --git a/src/server/Api/V1/Entries/CreateEntryRoute.cs b/src/server/Api/V1/Entries/CreateEntryRoute.cs new file mode 100644 index 0000000..ebe49fc --- /dev/null +++ b/src/server/Api/V1/Entries/CreateEntryRoute.cs @@ -0,0 +1,34 @@ +using IOL.BookmarkThing.Server.Api.V1.Entries.Dtos; + +namespace IOL.BookmarkThing.Server.Api.V1.Entries; + +public class CreateEntryRoute : RouteBaseV1Sync.WithRequest.WithActionResult +{ + private readonly AppDbContext _context; + + public CreateEntryRoute(AppDbContext context) { + _context = context; + } + + /// + /// Create a new entry + /// + /// The entry to create + /// Entry created successfully + /// Invalid entry + [ProducesResponseType(typeof(EntryDto), 200)] + [ProducesResponseType(typeof(List), 400)] + [ApiVersion(ApiSpecV1.VERSION_STRING)] + [HttpPost("~/v{version:apiVersion}/entries/create")] + public override ActionResult Handle(CreateEntryRequest entry) { + var errors = entry.GetErrors(); + if (errors.Count != 0) { + return BadRequest(errors); + } + + var dbEntry = entry.AsDbEntity(LoggedInUser.Id); + _context.Entries.Add(dbEntry); + _context.SaveChanges(); + return Ok(new EntryDto(dbEntry)); + } +} diff --git a/src/server/Api/V1/Entries/DeleteEntryRoute.cs b/src/server/Api/V1/Entries/DeleteEntryRoute.cs new file mode 100644 index 0000000..fc79049 --- /dev/null +++ b/src/server/Api/V1/Entries/DeleteEntryRoute.cs @@ -0,0 +1,30 @@ +namespace IOL.BookmarkThing.Server.Api.V1.Entries; + +public class DeleteEntryRoute : RouteBaseV1Sync.WithRequest.WithActionResult +{ + private readonly AppDbContext _context; + + public DeleteEntryRoute(AppDbContext context) { + _context = context; + } + + /// + /// Delete a entry + /// + /// The guid id of the entry to delete + /// Entry deleted successfully + /// Entry not found + [ProducesResponseType(typeof(ErrorResult), 404)] + [ApiVersion(ApiSpecV1.VERSION_STRING)] + [HttpDelete("~/v{version:apiVersion}/entries/{entryId:guid}")] + public override ActionResult Handle(Guid entryId) { + var entry = _context.Entries.SingleOrDefault(c => c.Id == entryId && c.UserId == LoggedInUser.Id); + if (entry == default) { + return NotFound(new ErrorResult("Entry does not exist")); + } + + _context.Remove(entry); + _context.SaveChanges(); + return Ok(); + } +} diff --git a/src/server/Api/V1/Entries/Dtos/EntryDto.cs b/src/server/Api/V1/Entries/Dtos/EntryDto.cs new file mode 100644 index 0000000..5161373 --- /dev/null +++ b/src/server/Api/V1/Entries/Dtos/EntryDto.cs @@ -0,0 +1,16 @@ +namespace IOL.BookmarkThing.Server.Api.V1.Entries.Dtos; + +public class EntryDto +{ + public EntryDto(Entry dbEntry) { + Id = dbEntry.Id; + Url = dbEntry.Url; + Description = dbEntry.Description; + Tags = dbEntry.Tags; + } + + public Guid Id { get; set; } + public Uri Url { get; set; } + public string Description { get; set; } + public string Tags { get; set; } +} diff --git a/src/server/Api/V1/Entries/GetEntriesRoute.cs b/src/server/Api/V1/Entries/GetEntriesRoute.cs new file mode 100644 index 0000000..adadf01 --- /dev/null +++ b/src/server/Api/V1/Entries/GetEntriesRoute.cs @@ -0,0 +1,21 @@ +using IOL.BookmarkThing.Server.Api.V1.Entries.Dtos; + +namespace IOL.BookmarkThing.Server.Api.V1.Entries; + +public class GetEntriesRoute : RouteBaseV1Sync.WithoutRequest.WithActionResult> +{ + private readonly AppDbContext _context; + + public GetEntriesRoute(AppDbContext context) { + _context = context; + } + + /// + /// Get all entries + /// + [ApiVersion(ApiSpecV1.VERSION_STRING)] + [HttpGet("~/v{version:apiVersion}/entries")] + public override ActionResult> Handle() { + return Ok(_context.Entries.Where(c => c.UserId == LoggedInUser.Id).Select(c => new EntryDto(c))); + } +} diff --git a/src/server/Api/V1/Entries/UpdateEntryRequest.cs b/src/server/Api/V1/Entries/UpdateEntryRequest.cs new file mode 100644 index 0000000..819e2d2 --- /dev/null +++ b/src/server/Api/V1/Entries/UpdateEntryRequest.cs @@ -0,0 +1,18 @@ +namespace IOL.BookmarkThing.Server.Api.V1.Entries; + +public class UpdateEntryRequest +{ + public Guid Id { get; set; } + public Uri Url { get; set; } + public string Description { get; set; } + public string Tags { get; set; } + + public List GetErrors() { + var errors = new List(); + if (Url == default) { + errors.Add(new ErrorResult("Url is a required field")); + } + + return errors; + } +} diff --git a/src/server/Api/V1/Entries/UpdateEntryRoute.cs b/src/server/Api/V1/Entries/UpdateEntryRoute.cs new file mode 100644 index 0000000..96c60fe --- /dev/null +++ b/src/server/Api/V1/Entries/UpdateEntryRoute.cs @@ -0,0 +1,45 @@ +using System.Security.Cryptography; +using IOL.BookmarkThing.Server.Api.V1.Entries.Dtos; + +namespace IOL.BookmarkThing.Server.Api.V1.Entries; + +public class UpdateEntryRoute : RouteBaseV1Sync.WithRequest.WithActionResult +{ + private readonly AppDbContext _context; + + public UpdateEntryRoute(AppDbContext context) { + _context = context; + } + + + /// + /// Update an entry + /// + /// The entry to update + /// Entry updated successfully + /// Invalid entry + [ProducesResponseType(typeof(EntryDto), 200)] + [ProducesResponseType(typeof(List), 400)] + [ProducesResponseType(typeof(ErrorResult), 404)] + [ApiVersion(ApiSpecV1.VERSION_STRING)] + [HttpPost("~/v{version:apiVersion}/entries/update")] + public override ActionResult Handle(UpdateEntryRequest entryToUpdate) { + var entry = _context.Entries.SingleOrDefault(c => c.Id == entryToUpdate.Id && c.UserId == LoggedInUser.Id); + if (entry == default) { + return NotFound(new ErrorResult("Entry does not exist")); + } + + var errors = entryToUpdate.GetErrors(); + if (errors.Count != 0) { + return BadRequest(errors); + } + + entry.Description = entry.Description; + entry.Tags = entry.Tags; + entry.Url = entry.Url; + + _context.Update(entry); + _context.SaveChanges(); + return Ok(new EntryDto(entry)); + } +} diff --git a/src/server/Api/V1/LoggedInV1User.cs b/src/server/Api/V1/LoggedInV1User.cs new file mode 100644 index 0000000..8c9f67a --- /dev/null +++ b/src/server/Api/V1/LoggedInV1User.cs @@ -0,0 +1,7 @@ +namespace IOL.BookmarkThing.Server.Api.V1; + +public class LoggedInV1User +{ + public Guid Id { get; set; } + public string Username { get; set; } +} diff --git a/src/server/Api/V1/RouteBaseV1Async.cs b/src/server/Api/V1/RouteBaseV1Async.cs new file mode 100644 index 0000000..d86bc7c --- /dev/null +++ b/src/server/Api/V1/RouteBaseV1Async.cs @@ -0,0 +1,73 @@ +namespace IOL.BookmarkThing.Server.Api.V1; + +/// +/// A base class for an endpoint that accepts parameters. +/// +public static class RouteBaseV1Async +{ + public static class WithRequest + { + public abstract class WithResult : BaseV1Route + { + public abstract Task HandleAsync( + TRequest request, + CancellationToken cancellationToken = default + ); + } + + public abstract class WithoutResult : BaseV1Route + { + public abstract Task HandleAsync( + TRequest request, + CancellationToken cancellationToken = default + ); + } + + public abstract class WithActionResult : BaseV1Route + { + public abstract Task> HandleAsync( + TRequest request, + CancellationToken cancellationToken = default + ); + } + + public abstract class WithActionResult : BaseV1Route + { + public abstract Task HandleAsync( + TRequest request, + CancellationToken cancellationToken = default + ); + } + } + + public static class WithoutRequest + { + public abstract class WithResult : BaseV1Route + { + public abstract Task HandleAsync( + CancellationToken cancellationToken = default + ); + } + + public abstract class WithoutResult : BaseV1Route + { + public abstract Task HandleAsync( + CancellationToken cancellationToken = default + ); + } + + public abstract class WithActionResult : BaseV1Route + { + public abstract Task> HandleAsync( + CancellationToken cancellationToken = default + ); + } + + public abstract class WithActionResult : BaseV1Route + { + public abstract Task HandleAsync( + CancellationToken cancellationToken = default + ); + } + } +} diff --git a/src/server/Api/V1/RouteBaseV1Sync.cs b/src/server/Api/V1/RouteBaseV1Sync.cs new file mode 100644 index 0000000..0bcc0ed --- /dev/null +++ b/src/server/Api/V1/RouteBaseV1Sync.cs @@ -0,0 +1,53 @@ +namespace IOL.BookmarkThing.Server.Api.V1; + +/// +/// A base class for an endpoint that accepts parameters. +/// +public static class RouteBaseV1Sync +{ + public static class WithRequest + { + public abstract class WithResult : BaseV1Route + { + public abstract TResponse Handle(TRequest request); + } + + public abstract class WithoutResult : BaseV1Route + { + public abstract void Handle(TRequest request); + } + + public abstract class WithActionResult : BaseV1Route + { + public abstract ActionResult Handle(TRequest request); + } + + public abstract class WithActionResult : BaseV1Route + { + public abstract ActionResult Handle(TRequest entry); + } + } + + public static class WithoutRequest + { + public abstract class WithResult : BaseV1Route + { + public abstract TResponse Handle(); + } + + public abstract class WithoutResult : BaseV1Route + { + public abstract void Handle(); + } + + public abstract class WithActionResult : BaseV1Route + { + public abstract ActionResult Handle(); + } + + public abstract class WithActionResult : BaseV1Route + { + public abstract ActionResult Handle(); + } + } +} -- cgit v1.3