Merge pull request #92 from stormpath/91-expand-groups

Add groups expansion back (closes #91)

src/Stormpath.Owin.Middleware/Model/MeGroupModel.cs

@@ -0,0 +1,19 @@
+using System;
+
+namespace Stormpath.Owin.Middleware.Model
+{
+    public sealed class MeGroupModel
+    {
+        public string Id { get; set; }
+
+        public string Name { get; set; }
+
+        public string Description { get; set; }
+
+        public string Status { get; } = "ENABLED"; // Groups in Okta are always enabled
+
+        public DateTimeOffset? CreatedAt { get; set; }
+
+        public DateTimeOffset? ModifiedAt { get; set; }
+    }
+}

src/Stormpath.Owin.Middleware/Model/MeGroupsCollectionModel.cs

@@ -0,0 +1,9 @@
+namespace Stormpath.Owin.Middleware.Model
+{
+    public sealed class MeGroupsCollectionModel
+    {
+        public int Size { get; set; }
+
+        public MeGroupModel[] Items { get; set; }
+    }
+}

src/Stormpath.Owin.Middleware/Model/MeResponseModel.cs

@@ -40,7 +40,7 @@ public sealed class MeResponseModel
         public IDictionary<string, object> CustomData { get; set; } = null;
 
         [JsonProperty(NullValueHandling = NullValueHandling.Ignore)]
-        public object Groups { get; set; } = null;
+        public MeGroupsCollectionModel Groups { get; set; } = null;
 
         [JsonProperty(NullValueHandling = NullValueHandling.Ignore)]
         public object ProviderData { get; set; } = null;

src/Stormpath.Owin.Middleware/Route/MeRoute.cs

@@ -41,14 +41,15 @@ protected override async Task<bool> GetJsonAsync(IOwinEnvironment context, Cance
 
             var responseModel = new
             {
-                account = await ExpandAccount(stormpathAccount, _configuration.Web.Me.Expand, cancellationToken)
+                account = await ExpandAccount(stormpathAccount, _oktaClient, _configuration.Web.Me.Expand, cancellationToken)
             };
 
             return await JsonResponse.Ok(context, responseModel);
         }
 
-        private static Task<MeResponseModel> ExpandAccount(
+        private static async Task<MeResponseModel> ExpandAccount(
             ICompatibleOktaAccount account,
+            IOktaClient oktaClient,
             IReadOnlyDictionary<string, bool> expansionOptions,
             CancellationToken cancellationToken)
         {
@@ -70,17 +71,36 @@ private static Task<MeResponseModel> ExpandAccount(
 
             if (!expansionOptions.Any(e => e.Value))
             {
-                return Task.FromResult(sanitizedModel);
+                return sanitizedModel;
             }
 
             if (expansionOptions.Any(e => e.Key.Equals("customData", StringComparison.OrdinalIgnoreCase) && e.Value))
             {
                 sanitizedModel.CustomData = account.CustomData;
             }
 
-            // TODO other expansion patches
+            if (expansionOptions.Any(e => e.Key.Equals("groups", StringComparison.OrdinalIgnoreCase) && e.Value))
+            {
+                var groups = await oktaClient.GetGroupsForUserIdAsync(account.GetOktaUser().Id, cancellationToken);
+
+                sanitizedModel.Groups = new MeGroupsCollectionModel
+                {
+                    Size = groups.Length,
+                    Items = groups.Select(g => new MeGroupModel
+                        {
+                            Id = g.Id,
+                            Name = g.Profile.Name,
+                            Description = g.Profile.Description,
+                            CreatedAt = g.Created,
+                            ModifiedAt = g.LastUpdated
+                        })
+                        .ToArray()
+                };
+            }
+
+            // TODO other expansion patches?
 
-            return Task.FromResult(sanitizedModel);
+            return sanitizedModel;
         }
     }
 }

src/Stormpath.Owin.Middleware/Stormpath.Owin.Middleware.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <Description>Stormpath OWIN middleware library</Description>
     <Copyright>(c) 2016 Stormpath, Inc.</Copyright>
-    <VersionPrefix>4.0.0-rc4</VersionPrefix>
+    <VersionPrefix>4.0.0-rc5</VersionPrefix>
     <Authors>Nate Barbettini</Authors>
     <TargetFrameworks>net451;netstandard1.4</TargetFrameworks>
     <NoWarn>$(NoWarn);CS1591;CS0618</NoWarn>