Catching SqlException does not work in my rest API

Question

Catching SqlException does not work in my rest API

Adagio_B 25

EDIT: Just delete this question. I currently don't have time to follow up

I'm having an issue, when my rest API code is deployed on our test server. I have the below code. This works fine on my development machine, but when deploying to our test-server and calling this method, all I'm getting is 500 internal error and the log file is not created. If I remove the part where it catches the SqlException, then calling the method works as expected and my log file is correctly created

I have installed the Nuget package System.Data.SqlClient 4.8.5

Does anybody have any idea what I am missing? From what I can tell, the System.Data.SqlClient.dll file on the server is the same as on my development machine

[HttpPut("Proposal/{labelId}/{buildId}/{identId}/{proposalId}")]
		public ResponseOpaqueConstruction AddProposal(int labelId, int buildId, int identId, int proposalId)
		{
			ResponseOpaqueConstruction response = new ResponseOpaqueConstruction();

			try
			{
				using (System.IO.StreamWriter file = new System.IO.StreamWriter(@"C:\Temp\TellMeSomething.log", true))
					file.WriteLine(DateTime.Now.ToString("T") + " - Inside try");
			}
			catch (SqlException ex)
			{
				
			}
			catch (Exception ex)
			{
				
			}

			return response;
		}

Jack J Jun 25,296 Reputation points

2023-05-01T08:59:29.8866667+00:00

@Adagio_B, Welcome to Microsoft Q&A, could you provide the specific message about SqlException, just 500 internal error? It seems that the code ResponseOpaqueConstruction response = new ResponseOpaqueConstruction(); throws the exception. How do you define the class ResponseOpaqueConstruction?
Adagio_B 25 Reputation points

2023-05-01T09:37:02.15+00:00
@Jack J Jun The method is not called at all if I include the catch (SqlException ex) line

ResponseOpaqueConstruction is just an class with a few simple variables, there's no code in there. As mentioned in OP, if I just remove these lines of code, then everything works fine

catch (SqlException ex) { }

But for my full code I do need to handle SqlExceptions differently than other exceptions. I removed all code that is not part of the problem, to simplify the actual problem

I am only getting an 500 internal error, no further details are sent to Postman as far as I can tell
Jack J Jun 25,296 Reputation points

2023-05-02T07:48:32.9333333+00:00
@Adagio_B, thanks for the feedback, if you want to catch the sqlexception and don't want to get the 500 internal error, do you consider the following workaround example for you?

try { // code here } catch (Exception ex) { if (ex is SqlException) { // Handle more specific SqlException exception here. } else { // Handle generic ones here. } }
Jack J Jun 25,296 Reputation points

2023-05-05T08:49:59.45+00:00

@Adagio_B, are you satisfied with my workround in the last comment?
Jack J Jun 25,296 Reputation points

2023-05-16T06:04:01.5133333+00:00

@Adagio_B, is any update for the question?

Your answer

Jack J Jun 25,296 Reputation points

2023-05-01T08:59:29.8866667+00:00

@Adagio_B, Welcome to Microsoft Q&A, could you provide the specific message about SqlException, just 500 internal error? It seems that the code ResponseOpaqueConstruction response = new ResponseOpaqueConstruction(); throws the exception. How do you define the class ResponseOpaqueConstruction?
Adagio_B 25 Reputation points

2023-05-01T09:37:02.15+00:00

@Jack J Jun The method is not called at all if I include the catch (SqlException ex) line

ResponseOpaqueConstruction is just an class with a few simple variables, there's no code in there. As mentioned in OP, if I just remove these lines of code, then everything works fine

catch (SqlException ex) { }

But for my full code I do need to handle SqlExceptions differently than other exceptions. I removed all code that is not part of the problem, to simplify the actual problem

I am only getting an 500 internal error, no further details are sent to Postman as far as I can tell
Jack J Jun 25,296 Reputation points

2023-05-02T07:48:32.9333333+00:00

@Adagio_B, thanks for the feedback, if you want to catch the sqlexception and don't want to get the 500 internal error, do you consider the following workaround example for you?

try { // code here } catch (Exception ex) { if (ex is SqlException) { // Handle more specific SqlException exception here. } else { // Handle generic ones here. } }
Jack J Jun 25,296 Reputation points

2023-05-05T08:49:59.45+00:00

@Adagio_B, are you satisfied with my workround in the last comment?
Jack J Jun 25,296 Reputation points

2023-05-16T06:04:01.5133333+00:00

@Adagio_B, is any update for the question?

Share via

Catching SqlException does not work in my rest API

Your answer