introduced logs and some conditions.

Nikita Kulshreshtha
1 parent a949d27f
Showing 12 changed files with 99 additions and 46 deletions
400-SOURCECODE/AIAHTML5.API/Constants/AIAConstants.cs
400-SOURCECODE/AIAHTML5.API/Controllers/ForgotUserController.cs
400-SOURCECODE/AIAHTML5.API/Models/ResetUser.cs
400-SOURCECODE/AIAHTML5.API/Models/Users.cs
400-SOURCECODE/AIAHTML5.API/Utility/EmailUtility.cs
400-SOURCECODE/AIAHTML5.API/Web.config
400-SOURCECODE/AIAHTML5.API/bin/AIAHTML5.API.dll
400-SOURCECODE/AIAHTML5.API/bin/AIAHTML5.API.dll.config
400-SOURCECODE/AIAHTML5.API/bin/AIAHTML5.API.pdb
400-SOURCECODE/AIAHTML5.Web/app/controllers/HomeController.js
400-SOURCECODE/AIAHTML5.Web/app/main/AIA.js
400-SOURCECODE/AIAHTML5.Web/index.html
@@ -8,8 +8,10 @@ namespace AIAHTML5.API.Constants
     public class AIAConstants
     {
        public  const string ERROR_IN_FECTHING_DETAILS = "Error in fecthing details.";
-
        public const string USER_NOT_FOUND = "User not found.";
+       public const string MAIL_NOT_SENT = "Mail not sent.";
+
+
     }
 }
 \ No newline at end of file
@@ -33,27 +33,40 @@ namespace AIAHTML5.API.Controllers
         //}
         public HttpResponseMessage Post([FromBody]JObject userInfo)
         {
-            ILog logger = log4net.LogManager.GetLogger((System.Reflection.MethodBase.GetCurrentMethod().DeclaringType));
-            logger.Debug("inside POST");
+            ILog logger = log4net.LogManager.GetLogger((System.Reflection.MethodBase.GetCurrentMethod().DeclaringType));
+            logger.Debug("inside POST in ForgotUserController for emailId = " + userInfo["emailId"]);
             dynamic userData = AIAHTML5.API.Models.Users.GetUserByEmail(userInfo);
             if (Convert.ToString(userData) != AIAConstants.USER_NOT_FOUND && Convert.ToString(userData) != AIAConstants.ERROR_IN_FECTHING_DETAILS)
-            {
+            {
+                logger.Debug("inside if in ForgotUserController userData.loginId= " + userData.loginId);
                 bool isMailSent = false;
-                string userDetails = Newtonsoft.Json.JsonConvert.SerializeObject(userData);
-                if (Convert.ToBoolean(userInfo["isPassword"]))
-                    isMailSent = AIAHTML5.API.Models.ResetUser.SendEmail(userData, true);
-                else
-                    isMailSent = AIAHTML5.API.Models.ResetUser.SendEmail(userData, false);
-
+                string userDetails = Newtonsoft.Json.JsonConvert.SerializeObject(userData);
+               
+                logger.Debug("1. inside if in ForgotUserController userDetails= " + userDetails);
+
+                if (Convert.ToBoolean(userInfo["isPassword"]))
+                {
+                    logger.Debug("2.  isPassword= " + Convert.ToBoolean(userInfo["isPassword"]));
+
+                    isMailSent = AIAHTML5.API.Models.ResetUser.SendEmail(userData, true);
+                }
+                else
+                {
+                    logger.Debug("3. isPassword= " + Convert.ToBoolean(userInfo["isPassword"]));
+
+                    isMailSent = AIAHTML5.API.Models.ResetUser.SendEmail(userData, false);
+                }
+                logger.Debug("isMailSent= " + isMailSent);
                 if (isMailSent)
                     return new HttpResponseMessage { StatusCode = HttpStatusCode.OK, Content = new StringContent(userDetails) };
                 else
-                    return new HttpResponseMessage { StatusCode = HttpStatusCode.InternalServerError, Content = new StringContent(userData) };
+                    return new HttpResponseMessage { StatusCode = HttpStatusCode.OK, Content = new StringContent(AIAConstants.MAIL_NOT_SENT)};
             }
             else
-            {
-                return new HttpResponseMessage { StatusCode = HttpStatusCode.OK, Content = new StringContent(userData) }; //new StringContent(userData)
+            {
+                logger.Debug("inside else in ForgotUserController userData= " + userData);
+                return new HttpResponseMessage { StatusCode = HttpStatusCode.OK, Content = new StringContent(userData)}; //new StringContent(userData)
             }
         }
@@ -12,14 +12,17 @@ using AIAHTML5.API.Utility;
 using System.Text;
 using System.IO;
 using System.Net.Mime;
-using System.Configuration;
+using System.Configuration;
 namespace AIAHTML5.API.Models
 {
     public class ResetUser
     {
         public static bool SendEmail(dynamic UserDetails, bool isPassword)
-        {
+        {
+            ILog logger = log4net.LogManager.GetLogger((System.Reflection.MethodBase.GetCurrentMethod().DeclaringType));
+            logger.Debug("inside SendEmail in  for isPassword =" + isPassword);
+            
             try
             {
                 List<LinkedResource> lstLinkedResource = new List<LinkedResource>();
@@ -63,15 +66,16 @@ namespace AIAHTML5.API.Models
                     resetPasswordLink = site_url + "?em:" + HttpUtility.UrlEncode(userMail);
                 }
                 else
-                    templatePath = "~/Templates/forgot-UserId.html";
-
+                    templatePath = "~/Templates/forgot-UserId.html";
+
+                logger.Debug("inside SendEmail for templatePath= " + templatePath + ", userId= " + userId + ", userMail= " + userMail + ",fullName= " + fullName + ",resetPasswordLink= " + resetPasswordLink);
                 string mailBody = ResetUser.GetMailBodyTextFromTemplate(templatePath, userId, userMail, fullName, resetPasswordLink);
                 lstToAddress.Add(userMail);
-                emailText = mailBody;
-
-
+                emailText = mailBody;
+
+                logger.Debug("emailText= " + emailText);
                 // for embedding images in email
                 if (emailText.Contains("<img"))
                 {
@@ -92,9 +96,9 @@ namespace AIAHTML5.API.Models
                 if(!isPassword)
                     mailSubject = "UserID recovery mail for: ";
                 else
-                    mailSubject = "Password recovery mail for: ";                
-
-                emailUtility.sHostName = "10.100.12.13";
+                    mailSubject = "Password recovery mail for: ";
+
+                emailUtility.sHostName = Convert.ToString(ConfigurationManager.AppSettings["HostAddress"]);
                 emailUtility.sFromAddress = Convert.ToString(ConfigurationManager.AppSettings["SenderEmailAddress"]);
                 emailUtility.bIsBodyHtml = true;
                 emailUtility.bEnableSsl = false;
@@ -115,7 +119,10 @@ namespace AIAHTML5.API.Models
         }
         protected static string GetMailBodyTextFromTemplate(string templatePath, string userId, string userMail, string fullName, string resetPasswordUrl)
-        {
+        {
+            ILog logger = log4net.LogManager.GetLogger((System.Reflection.MethodBase.GetCurrentMethod().DeclaringType));
+            logger.Debug("inside GetMailBodyTextFromTemplate for templatePath= " + templatePath + ", userId= " + userId + ", userMail= " + userMail + ",fullName= " + fullName + ",resetPasswordLink= " + resetPasswordUrl);
+            
             string fileToRead = string.Empty;
             string emailBody = string.Empty;
             try
@@ -138,7 +145,8 @@ namespace AIAHTML5.API.Models
             }
             catch (Exception e)
             {
-                emailBody = "Exception: " + e.Message;
+                emailBody = "Exception: " + e.Message;
+                logger.Fatal("exception in GetMailBodyTextFromTemplate. msg= " + e.Message + ", stacktrace= " + e.StackTrace);
             }
             return emailBody;
         }
@@ -55,6 +55,7 @@ namespace AIAHTML5.API.Models
         internal static dynamic GetUserByEmail(Newtonsoft.Json.Linq.JObject userInfo)
         {
             ILog logger = log4net.LogManager.GetLogger((System.Reflection.MethodBase.GetCurrentMethod().DeclaringType));
+            logger.Debug(" inside GetUserByEmail for emailId = " + userInfo["emailId"] );
             var client = new MongoClient();
             try
@@ -67,15 +68,21 @@ namespace AIAHTML5.API.Models
                 FilterDefinition<dynamic> filterCondition = Builders<dynamic>.Filter.Eq("emailId", userInfo["emailId"].ToString().ToLower());
                 dynamic userDetails = collection.Find(Builders<dynamic>.Filter.And(filterCondition)).SingleOrDefault();
-
                 if (userDetails != null)
+                {
+                    logger.Debug("userDetails= " + userDetails.loginId);
+
                     return userDetails;
+                }
                 else
+                {
+                    logger.Debug("user not found");
                     return AIAConstants.USER_NOT_FOUND;
+                }
             }
             catch (Exception e)
             {
-                logger.Fatal("Exception= " + e.Message);
+                logger.Fatal("Exception= " + e.Message+", stacktrace = "+e.StackTrace);
                 return AIAConstants.ERROR_IN_FECTHING_DETAILS;
             }
         }
@@ -7,7 +7,8 @@ using System.Collections;
 using System.Xml;
 using System.Text;
 using System.IO;
-using System.Net.Mail;
+using System.Net.Mail;
+using log4net;
 namespace AIAHTML5.API.Utility
 {
@@ -41,7 +42,10 @@ namespace AIAHTML5.API.Utility
         }
         public void SendSmtpEmail()
-        {
+        {
+            ILog logger = log4net.LogManager.GetLogger((System.Reflection.MethodBase.GetCurrentMethod().DeclaringType));
+            logger.Debug("inside SendSmtpEmail");
+            
             try
             {
                 MailMessage sMail = new MailMessage();
@@ -53,8 +57,8 @@ namespace AIAHTML5.API.Utility
                 {
                     foreach (var sItem in sToAddresses)
                     {
-                        sMail.To.Add(sItem);
-
+                        sMail.To.Add(sItem);
+                        logger.Debug("sToAddresses= " + sItem);
                     }
                 }
@@ -62,7 +66,9 @@ namespace AIAHTML5.API.Utility
                 {
                     foreach (var sItem in sBccAddresses)
                     {
-                        sMail.Bcc.Add(sItem);
+                        sMail.Bcc.Add(sItem);
+                        logger.Debug("sBccAddresses= " + sItem);
+
                     }
                 }
@@ -70,21 +76,31 @@ namespace AIAHTML5.API.Utility
                 if (sAlternateView != null)
                 {
-                    sMail.AlternateViews.Add(sAlternateView);
+                    sMail.AlternateViews.Add(sAlternateView);
+                    logger.Debug("sAlternateView= " + sAlternateView);
+
                 }
                 else
                 {
-                    sMail.Body = sBodyText;
+                    sMail.Body = sBodyText;
+                    logger.Debug("sMail.Body= " + sBodyText);
+
                 }
-                sMail.Subject = sSubject;
+                sMail.Subject = sSubject;
+                logger.Debug("sMail.Subject= " + sSubject);
+
                 if (sAttachments != null)
                 {
                     foreach (var sItem in sAttachments)
                     {
-                        sMail.Attachments.Add(sItem);
+                        sMail.Attachments.Add(sItem);
+                        logger.Debug("sAttachments= " + sAttachments);
+
+
                     }
-                }
+                }
+                logger.Debug("sUserName= " + sUserName + ", sPassword= " + sPassword);
                 SmtpServer.Port = iPort;
                 SmtpServer.Credentials = new System.Net.NetworkCredential(sUserName, sPassword);
@@ -105,12 +121,14 @@ namespace AIAHTML5.API.Utility
                     }
                     mm.IsBodyHtml = true;
-                    SendMail(mm);
+                    SendMail(mm);
+                    logger.Debug("after sending email");
                 }
             }
             catch (Exception ex)
-            {
+            {
+                logger.Fatal("exception in SendSmtpEmail.msg= " + ex.Message + ", stacktarce= " + ex.StackTrace);
                 throw ex;
             }
         }
@@ -37,8 +37,8 @@
     <add key="SMTPAddress" value="smtp.emailsrvr.com" />
     <add key="SMTPPort" value="587" />
     <add key="EnableSSL" value="false" />
-    <add key="NotifyEmailAddress" value="support@interactiveanatomy.com" />
     <add key="Site_Url" value ="//52.2.38.120"/>
+    <add key ="HostAddress" value="10.100.12.13" />
   </appSettings>
   <system.web>
     <compilation debug="true" targetFramework="4.5" />
@@ -39,6 +39,7 @@
     <add key="EnableSSL" value="false" />
     <add key="NotifyEmailAddress" value="support@interactiveanatomy.com" />
     <add key="Site_Url" value ="//52.2.38.120"/>
+    <add key ="HostAddress" value="10.100.12.13" />
   </appSettings>
   <system.web>
     <compilation debug="true" targetFramework="4.5" />
@@ -48,7 +49,7 @@
   <applicationSettings>
     <AIAHTML5.API.Properties.Settings>
       <setting name="AIAConnectionString" serializeAs="String">
-        <value>192.168.90.157:27017</value>
+        <value>localhost:27017</value>
       </setting>
       <setting name="database" serializeAs="String">
         <value>AIA</value>
@@ -179,7 +179,7 @@ function ($rootScope, Modules, $log, $location, $timeout, DataService, Authentic
     $rootScope.SendMailToUser = function (userInfo, isMailForPassword) {
         if ((userInfo.emailId != null) && (userInfo.emailId != '')) {
             if (validateEmail(userInfo.emailId)) {
-                if (isMailForPassword == 'true')
+                if (isMailForPassword == true)
                     userInfo.isPassword = true;
                 else
                     userInfo.isPassword = false;
@@ -187,10 +187,13 @@ function ($rootScope, Modules, $log, $location, $timeout, DataService, Authentic
                 AuthenticationService.SendMailToUser(userInfo)
                 .then(function (result) {
                     if (result == LoginConstants.USER_NOT_FOUND) {
-                        alert("Error occured.");
+                        alert(LoginConstants.USER_NOT_FOUND);
                     }
                     else if (result == LoginConstants.ERROR_IN_FECTHING_DETAILS) {
-                        alert("Error occured.");
+                        alert("An error occured.");
+                    }
+                    else if (result == LoginConstants.MAIL_NOT_SENT) {
+                        alert("Due to some issue, email not sent.");
                     }
                     else {
                         if (result.loginId != undefined || result.loginId != "" || result.loginId != null) {
@@ -283,6 +283,7 @@ AIA.constant(&#39;ImageTypes&#39;, [&#39;Illustration&#39;, &#39;Cadaver Photograph&#39;, &#39;Radiograph&#39;])
 AIA.constant("LoginConstants", {
     "USER_NOT_FOUND": "User not found.",
     "ERROR_IN_FECTHING_DETAILS": "Error in fecthing details.",
+    "MAIL_NOT_SENT" : "Mail not sent.",
 })
@@ -243,7 +243,7 @@
                         </form>
                     </div>
                     <div class="modal-footer">
-                        <button type="button" class="btn btn-primary btn-block" ng-click="SendMailToUser(userInfo, 'false')">Send Mail</button>
+                        <button type="button" class="btn btn-primary btn-block" ng-click="SendMailToUser(userInfo, false)">Send Mail</button>
                     </div>
                     <!--<div style="color: maroon; font-weight: bold; " ng-if="message">{{message}}</div>-->
                 </div>
@@ -268,7 +268,7 @@
                         </form>
                     </div>
                     <div class="modal-footer">
-                        <button type="button" class="btn btn-primary btn-block" ng-click="SendMailToUser(userInfo, 'true')">Send Mail</button>
+                        <button type="button" class="btn btn-primary btn-block" ng-click="SendMailToUser(userInfo, true)">Send Mail</button>
                     </div>
                     <!--<div style="color: maroon; font-weight: bold; " ng-if="message">{{message}}</div>-->
                 </div>