Merge remote-tracking branch 'upstream/DotNetCore' into DotNetCore

pull/1488/head
Dhruv Bhavsar 7 years ago
commit 4e138b948e

@ -1,6 +1,6 @@
version: 3.0.{build}
configuration: Release
os: Visual Studio 2017
os: Visual Studio 2015
environment:
nodejs_version: "7.8.0"
@ -8,7 +8,7 @@ install:
# Get the latest stable version of Node.js or io.js
- ps: Install-Product node $env:nodejs_version
build_script:
- ps: ./build.ps1
- ps: ./build.ps1 -Verbosity Diagnostic
after_build:
- cmd: >-

@ -94,6 +94,7 @@ namespace Ombi.DependencyInjection
services.AddTransient<IMovieRequestRepository, MovieRequestRepository>();
services.AddTransient<IAuditRepository, AuditRepository>();
services.AddTransient<IApplicationConfigRepository, ApplicationConfigRepository>();
services.AddTransient<ITokenRepository, TokenRepository>();
services.AddTransient(typeof(ISettingsService<>), typeof(SettingsService<>));
}
public static void RegisterServices(this IServiceCollection services)

@ -29,5 +29,6 @@ namespace Ombi.Store.Context
DbSet<ChildRequests> ChildRequests { get; set; }
DbSet<MovieIssues> MovieIssues { get; set; }
DbSet<TvIssues> TvIssues { get; set; }
DbSet<Tokens> Tokens { get; set; }
}
}

@ -35,6 +35,7 @@ namespace Ombi.Store.Context
public DbSet<TvIssues> TvIssues { get; set; }
public DbSet<Audit> Audit { get; set; }
public DbSet<Tokens> Tokens { get; set; }
public DbSet<ApplicationConfiguration> ApplicationConfigurations { get; set; }

@ -0,0 +1,14 @@
using System.ComponentModel.DataAnnotations.Schema;
namespace Ombi.Store.Entities
{
public class Tokens : Entity
{
public string Token { get; set; }
public string UserId { get; set; }
[ForeignKey(nameof(UserId))]
public OmbiUser User { get; set; }
}
}

@ -10,7 +10,7 @@ using Ombi.Helpers;
namespace Ombi.Store.Migrations
{
[DbContext(typeof(OmbiContext))]
[Migration("20170801143617_Inital")]
[Migration("20170811145836_Inital")]
partial class Inital
{
protected override void BuildTargetModel(ModelBuilder modelBuilder)
@ -449,6 +449,22 @@ namespace Ombi.Store.Migrations
b.ToTable("TvRequests");
});
modelBuilder.Entity("Ombi.Store.Entities.Tokens", b =>
{
b.Property<int>("Id")
.ValueGeneratedOnAdd();
b.Property<string>("Token");
b.Property<string>("UserId");
b.HasKey("Id");
b.HasIndex("UserId");
b.ToTable("Tokens");
});
modelBuilder.Entity("Ombi.Store.Repository.Requests.EpisodeRequests", b =>
{
b.Property<int>("Id")
@ -581,6 +597,13 @@ namespace Ombi.Store.Migrations
.OnDelete(DeleteBehavior.Cascade);
});
modelBuilder.Entity("Ombi.Store.Entities.Tokens", b =>
{
b.HasOne("Ombi.Store.Entities.OmbiUser", "User")
.WithMany()
.HasForeignKey("UserId");
});
modelBuilder.Entity("Ombi.Store.Repository.Requests.EpisodeRequests", b =>
{
b.HasOne("Ombi.Store.Repository.Requests.SeasonRequests", "Season")

@ -296,6 +296,26 @@ namespace Ombi.Store.Migrations
onDelete: ReferentialAction.Restrict);
});
migrationBuilder.CreateTable(
name: "Tokens",
columns: table => new
{
Id = table.Column<int>(nullable: false)
.Annotation("Sqlite:Autoincrement", true),
Token = table.Column<string>(nullable: true),
UserId = table.Column<string>(nullable: true)
},
constraints: table =>
{
table.PrimaryKey("PK_Tokens", x => x.Id);
table.ForeignKey(
name: "FK_Tokens_AspNetUsers_UserId",
column: x => x.UserId,
principalTable: "AspNetUsers",
principalColumn: "Id",
onDelete: ReferentialAction.Restrict);
});
migrationBuilder.CreateTable(
name: "PlexSeasonsContent",
columns: table => new
@ -531,6 +551,11 @@ namespace Ombi.Store.Migrations
table: "TvIssues",
column: "TvId");
migrationBuilder.CreateIndex(
name: "IX_Tokens_UserId",
table: "Tokens",
column: "UserId");
migrationBuilder.CreateIndex(
name: "IX_EpisodeRequests_SeasonId",
table: "EpisodeRequests",
@ -583,6 +608,9 @@ namespace Ombi.Store.Migrations
migrationBuilder.DropTable(
name: "TvIssues");
migrationBuilder.DropTable(
name: "Tokens");
migrationBuilder.DropTable(
name: "EpisodeRequests");

@ -448,6 +448,22 @@ namespace Ombi.Store.Migrations
b.ToTable("TvRequests");
});
modelBuilder.Entity("Ombi.Store.Entities.Tokens", b =>
{
b.Property<int>("Id")
.ValueGeneratedOnAdd();
b.Property<string>("Token");
b.Property<string>("UserId");
b.HasKey("Id");
b.HasIndex("UserId");
b.ToTable("Tokens");
});
modelBuilder.Entity("Ombi.Store.Repository.Requests.EpisodeRequests", b =>
{
b.Property<int>("Id")
@ -580,6 +596,13 @@ namespace Ombi.Store.Migrations
.OnDelete(DeleteBehavior.Cascade);
});
modelBuilder.Entity("Ombi.Store.Entities.Tokens", b =>
{
b.HasOne("Ombi.Store.Entities.OmbiUser", "User")
.WithMany()
.HasForeignKey("UserId");
});
modelBuilder.Entity("Ombi.Store.Repository.Requests.EpisodeRequests", b =>
{
b.HasOne("Ombi.Store.Repository.Requests.SeasonRequests", "Season")

@ -1,12 +1,13 @@
using System;
using System.Linq;
using System.Threading.Tasks;
using Ombi.Store.Entities;
namespace Ombi.Store.Repository
{
//public interface ITokenRepository
//{
// Task CreateToken(EmailTokens token);
// Task<EmailTokens> GetToken(Guid tokenId);
//}
public interface ITokenRepository
{
Task CreateToken(Tokens token);
IQueryable<Tokens> GetToken(string tokenId);
}
}

@ -2,29 +2,29 @@
using Ombi.Store.Context;
using Ombi.Store.Entities;
using System;
using System.Linq;
using System.Threading.Tasks;
namespace Ombi.Store.Repository
{
//public class TokenRepository : ITokenRepository
//{
// public TokenRepository(IOmbiContext db)
// {
// Db = db;
// }
public class TokenRepository : ITokenRepository
{
public TokenRepository(IOmbiContext db)
{
Db = db;
}
// private IOmbiContext Db { get; }
private IOmbiContext Db { get; }
// public async Task CreateToken(EmailTokens token)
// {
// token.Token = Guid.NewGuid();
// await Db.EmailTokens.AddAsync(token);
// await Db.SaveChangesAsync();
// }
public async Task CreateToken(Tokens token)
{
await Db.Tokens.AddAsync(token);
await Db.SaveChangesAsync();
}
// public async Task<EmailTokens> GetToken(Guid tokenId)
// {
// return await Db.EmailTokens.FirstOrDefaultAsync(x => x.Token == tokenId);
// }
//}
public IQueryable<Tokens> GetToken(string tokenId)
{
return Db.Tokens.Where(x => x.Token == tokenId);
}
}
}

@ -20,18 +20,20 @@ namespace Ombi.Controllers
public class TokenController
{
public TokenController(UserManager<OmbiUser> um, IOptions<TokenAuthentication> ta,
IApplicationConfigRepository config, IAuditRepository audit)
IApplicationConfigRepository config, IAuditRepository audit, ITokenRepository token)
{
UserManager = um;
TokenAuthenticationOptions = ta.Value;
Config = config;
Audit = audit;
_userManager = um;
_tokenAuthenticationOptions = ta.Value;
_config = config;
_audit = audit;
_token = token;
}
private TokenAuthentication TokenAuthenticationOptions { get; }
private IApplicationConfigRepository Config { get; }
private IAuditRepository Audit { get; }
private UserManager<OmbiUser> UserManager { get; }
private readonly TokenAuthentication _tokenAuthenticationOptions;
private IApplicationConfigRepository _config;
private readonly IAuditRepository _audit;
private readonly ITokenRepository _token;
private readonly UserManager<OmbiUser> _userManager;
/// <summary>
/// Gets the token.
@ -41,40 +43,31 @@ namespace Ombi.Controllers
[HttpPost]
public async Task<IActionResult> GetToken([FromBody] UserAuthModel model)
{
await Audit.Record(AuditType.None, AuditArea.Authentication,
await _audit.Record(AuditType.None, AuditArea.Authentication,
$"Username {model.Username} attempting to authenticate");
var user = await UserManager.FindByNameAsync(model.Username);
var user = await _userManager.FindByNameAsync(model.Username);
if (user == null)
{
return null;
return new UnauthorizedResult();
}
// Verify Password
if ((await UserManager.CheckPasswordAsync(user, model.Password)))
if (await _userManager.CheckPasswordAsync(user, model.Password))
{
// Get the url
var url = Config.Get(ConfigurationTypes.Url);
var port = Config.Get(ConfigurationTypes.Port);
#if !DEBUG
var audience = $"{url}:{port}";
#else
var audience = $"http://localhost:52038/";
#endif
var roles = await UserManager.GetRolesAsync(user);
var roles = await _userManager.GetRolesAsync(user);
var claims = new List<Claim>
{
new Claim(JwtRegisteredClaimNames.Sub, user.UserName),
new Claim("name", user.UserName),
new Claim(ClaimTypes.NameIdentifier, user.Id),
new Claim(ClaimTypes.Name, user.UserName),
new Claim(JwtRegisteredClaimNames.Jti, Guid.NewGuid().ToString())
};
claims.AddRange(roles.Select(role => new Claim("role", role)));
var key = new SymmetricSecurityKey(Encoding.UTF8.GetBytes(TokenAuthenticationOptions.SecretKey));
var key = new SymmetricSecurityKey(Encoding.UTF8.GetBytes(_tokenAuthenticationOptions.SecretKey));
var creds = new SigningCredentials(key, SecurityAlgorithms.HmacSha256);
var token = new JwtSecurityToken(
@ -83,15 +76,21 @@ namespace Ombi.Controllers
signingCredentials: creds,
audience: "Ombi", issuer:"Ombi"
);
var accessToken = new JwtSecurityTokenHandler().WriteToken(token);
if (model.RememberMe)
{
// Save the token so we can refresh it later
await _token.CreateToken(new Tokens() {Token = accessToken, User = user});
}
return new JsonResult(new
{
access_token = new JwtSecurityTokenHandler().WriteToken(token),
access_token = accessToken,
expiration = token.ValidTo
});
}
return null;
return new UnauthorizedResult();
}
/// <summary>
@ -101,10 +100,25 @@ namespace Ombi.Controllers
/// <returns></returns>
/// <exception cref="NotImplementedException"></exception>
[HttpPost("refresh")]
public async Task<IActionResult> RefreshToken([FromBody] UserAuthModel model)
public async Task<IActionResult> RefreshToken([FromBody] TokenRefresh token)
{
// Check if token exists
var dbToken = _token.GetToken(token.Token).FirstOrDefault();
if (dbToken == null)
{
return new UnauthorizedResult();
}
throw new NotImplementedException();
}
public class TokenRefresh
{
public string Token { get; set; }
public string Userename { get; set; }
}
}
}

@ -4,5 +4,6 @@
{
public string Username { get; set; }
public string Password { get; set; }
public bool RememberMe { get; set; }
}
}
Loading…
Cancel
Save