Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

STOP-2386 #2679

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sohitSmartbear
Copy link

Addresses #STOP-2386

Summary

Critical Logging Issue with Prism Proxy

Current Behavior
Attached are our config, a video walking through how we produced the issue locally and the smallest JSON body that we could reproduce the issue. (pügememory) in the JSON contains ü character we found during testing.

Notes

We were only ever able to reproduce this with a large payload that contains the ü character.

During the debugging process we noticed the error being thrown around a missing “:” from what we could see.

In some cases, different sized requests (smaller and larger) would not cause the issue. We would insert duplicate in the attributes section like in the JSON example.

Below are the steps we used for reproducing this locally:

Running Prism Proxy Locally with our yml config
Checklist

  • The basics
    • I tested these changes manually in my local or dev environment
  • Tests
    • Added or updated
    • N/A
  • Event Tracking
    • I added event tracking and followed the event tracking guidelines
    • N/A
  • Error Reporting
    • I reported errors and followed the error reporting guidelines
    • N/A

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant