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

fix: capture redirected requests in waterfall #180

Merged
merged 2 commits into from Jan 12, 2021

Conversation

vigneshshanmugam
Copy link
Member

  • fix Redirects and DNS failures not represented correctly #175
  • Redirected requests have the same requestId which was overriding the original request with the redirected request. As a result the redirected requests was simply not shown in the waterfall.
  • The solution is to check if the request is issued with the same requestId and simply mimic the usual flow from requestSent -> responseReceived -> LoadingFinished by using the redirected response which is available as part of the event.redirectedResponse data.

Check the below events for a sample website which goes from HTTP -> HTTPS and how 301 works.

{"type":"journey/network_info","@timestamp":1609895099415610.8,"journey":{"name":"check that title is present","id":"check that title is present"},"step":{"name":"go to app","index":1},"payload":{"step":{"name":"go to app","index":1},"timestamp":1609895099415610.8,"url":"http://vigneshh.in/","request":{"url":"http://vigneshh.in/","method":"GET","headers":{"upgrade_insecure_requests":"1","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.0 Safari/537.36","host":"vigneshh.in","connection":"keep-alive","accept":"text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9","accept_encoding":"gzip, deflate","accept_language":"en-GB,en;q=0.9"},"mixed_content_type":"none","initial_priority":"VeryHigh","referrer_policy":"strict-origin-when-cross-origin"},"type":"Document","method":"GET","request_sent_time":1012040.277427,"is_navigation_request":true,"status":301,"load_end_time":1012040.365573,"response_received_time":1012040.365573,"response":{"url":"http://vigneshh.in/","status":301,"status_text":"Moved Permanently","headers":{"date":"Wed, 06 Jan 2021 01:04:59 GMT","transfer_encoding":"chunked","connection":"keep-alive","cache_control":"max-age=3600","expires":"Wed, 06 Jan 2021 02:04:59 GMT","location":"https://vigneshh.in/","cf_request_id":"0776d2d4760000f55947917000000001","report_to":"{\"endpoints\":[{\"url\":\"https:\\/\\/a.nel.cloudflare.com\\/report?s=ETVgpWk5MwNmSv7ZSCQ%2B%2F5mWTTcwMGltJh0waEBoj%2FNrzwHHHikgeL5fas7I5zXcmBhvy7DOshY0K%2B7XUlaGOjpzM80ZK6s0AHCB1ptbObHBLe%2FGsWFNgQ%3D%3D\"}],\"group\":\"cf-nel\",\"max_age\":604800}","nel":"{\"report_to\":\"cf-nel\",\"max_age\":604800}","vary":"Accept-Encoding","x_content_type_options":"nosniff","server":"cloudflare","cf_ray":"60d18733f9e9f559-SEA"},"headers_text":"HTTP/1.1 301 Moved Permanently\r\nDate: Wed, 06 Jan 2021 01:04:59 GMT\r\nTransfer-Encoding: chunked\r\nConnection: keep-alive\r\nCache-Control: max-age=3600\r\nExpires: Wed, 06 Jan 2021 02:04:59 GMT\r\nLocation: https://vigneshh.in/\r\ncf-request-id: 0776d2d4760000f55947917000000001\r\nReport-To: {\"endpoints\":[{\"url\":\"https:\\/\\/a.nel.cloudflare.com\\/report?s=ETVgpWk5MwNmSv7ZSCQ%2B%2F5mWTTcwMGltJh0waEBoj%2FNrzwHHHikgeL5fas7I5zXcmBhvy7DOshY0K%2B7XUlaGOjpzM80ZK6s0AHCB1ptbObHBLe%2FGsWFNgQ%3D%3D\"}],\"group\":\"cf-nel\",\"max_age\":604800}\r\nNEL: {\"report_to\":\"cf-nel\",\"max_age\":604800}\r\nVary: Accept-Encoding\r\nX-Content-Type-Options: nosniff\r\nServer: cloudflare\r\nCF-RAY: 60d18733f9e9f559-SEA\r\n\r\n","mime_type":"","request_headers":{"host":"vigneshh.in","connection":"keep-alive","upgrade_insecure_requests":"1","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.0 Safari/537.36","accept":"text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9","accept_encoding":"gzip, deflate","accept_language":"en-GB,en;q=0.9"},"request_headers_text":"GET / HTTP/1.1\r\nHost: vigneshh.in\r\nConnection: keep-alive\r\nUpgrade-Insecure-Requests: 1\r\nUser-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.0 Safari/537.36\r\nAccept: text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9\r\nAccept-Encoding: gzip, deflate\r\nAccept-Language: en-GB,en;q=0.9\r\n","connection_reused":false,"connection_id":52,"remote_i_p_address":"[2606:4700:3030::681c:133e]","remote_port":80,"from_disk_cache":false,"from_service_worker":false,"from_prefetch_cache":false,"encoded_data_length":673,"timing":{"request_time":1012040.278533,"proxy_start":-1,"proxy_end":-1,"dns_start":0.347,"dns_end":35.274,"connect_start":35.274,"connect_end":50.998,"ssl_start":-1,"ssl_end":-1,"worker_start":-1,"worker_ready":-1,"worker_fetch_start":-1,"worker_respond_with_settled":-1,"send_start":51.102,"send_end":51.187,"push_start":0,"push_end":0,"receive_headers_end":85.369},"response_time":1609895099499.776,"protocol":"http/1.1","security_state":"insecure"},"timings":{"blocked":0.3469999646767974,"queueing":1.1060000397264957,"proxy":-1,"dns":34.92700005881488,"ssl":-1,"connect":15.72399993892759,"send":0.0849999487400055,"wait":34.18199997395277,"receive":1.6710000345483422,"total":88.14600005280226}},"package_version":"0.0.1-alpha.8"}

{"type":"journey/network_info","@timestamp":1609895099505859,"journey":{"name":"check that title is present","id":"check that title is present"},"step":{"name":"go to app","index":1},"payload":{"step":{"name":"go to app","index":1},"timestamp":1609895099505859,"url":"https://vigneshh.in/","request":{"url":"https://vigneshh.in/","method":"GET","headers":{"upgrade_insecure_requests":"1","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.0 Safari/537.36","method":"GET","authority":"vigneshh.in","scheme":"https","path":"/","accept":"text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9","sec_fetch_site":"none","sec_fetch_mode":"navigate","sec_fetch_user":"?1","sec_fetch_dest":"document","sec_ch_ua":"\";Not\\\\A\\\"Brand\";v=\"99\", \"Chromium\";v=\"88\"","sec_ch_ua_mobile":"?0","accept_encoding":"gzip, deflate, br","accept_language":"en-GB,en;q=0.9"},"mixed_content_type":"none","initial_priority":"VeryHigh","referrer_policy":"strict-origin-when-cross-origin"},"type":"Document","method":"GET","request_sent_time":1012040.365573,"is_navigation_request":true,"status":200,"load_end_time":1012040.564763,"response_received_time":1012040.567352,"response":{"url":"https://vigneshh.in/","status":200,"status_text":"","headers":{"date":"Wed, 06 Jan 2021 01:04:59 GMT","content_type":"text/html; charset=utf-8","set_cookie":"__cfduid=df3f740b1d7c6b826023178e9eb075e731609895099; expires=Fri, 05-Feb-21 01:04:59 GMT; path=/; domain=.vigneshh.in; HttpOnly; SameSite=Lax","last_modified":"Wed, 17 Jun 2020 21:25:03 GMT","access_control_allow_origin":"*","expires":"Tue, 05 Jan 2021 23:33:29 GMT","cache_control":"max-age=600","x_proxy_cache":"MISS","x_github_request_id":"3A50:9DDA:1D63276:217135C:5FF4F4F0","via":"1.1 varnish","age":"0","x_served_by":"cache-sea4480-SEA","x_cache":"HIT","x_cache_hits":"1","x_timer":"S1609895100.637506,VS0,VE77","vary":"Accept-Encoding","x_fastly_request_id":"530154c96c465eb52b011e3840dd4abdbe458cec","cf_cache_status":"DYNAMIC","cf_request_id":"0776d2d4ec0000f595ecb0c000000001","expect_ct":"max-age=604800, report-uri=\"https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct\"","report_to":"{\"endpoints\":[{\"url\":\"https:\\/\\/a.nel.cloudflare.com\\/report?s=iGmKNu4iAQOZ%2BSHSlaXGYbHoyQpRBfjQPT91NCwk75y%2B1K%2BoOVynIlQJJkh0UfWhCLB0Had1oyismCAri9wwwJEsteUB79Bc%2Bw%2BZfTREevWB0eJYvA5yvQ%3D%3D\"}],\"group\":\"cf-nel\",\"max_age\":604800}","nel":"{\"report_to\":\"cf-nel\",\"max_age\":604800}","strict_transport_security":"max-age=2592000; includeSubDomains","x_content_type_options":"nosniff","server":"cloudflare","cf_ray":"60d18734aee8f595-SEA","content_encoding":"br"},"mime_type":"text/html","request_headers":{"method":"GET","authority":"vigneshh.in","scheme":"https","path":"/","upgrade_insecure_requests":"1","user_agent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.0 Safari/537.36","accept":"text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9","sec_fetch_site":"none","sec_fetch_mode":"navigate","sec_fetch_user":"?1","sec_fetch_dest":"document","sec_ch_ua":"\";Not\\\\A\\\"Brand\";v=\"99\", \"Chromium\";v=\"88\"","sec_ch_ua_mobile":"?0","accept_encoding":"gzip, deflate, br","accept_language":"en-GB,en;q=0.9"},"connection_reused":false,"connection_id":62,"remote_i_p_address":"[2606:4700:3030::681c:133e]","remote_port":443,"from_disk_cache":false,"from_service_worker":false,"from_prefetch_cache":false,"encoded_data_length":3332,"timing":{"request_time":1012040.367565,"proxy_start":-1,"proxy_end":-1,"dns_start":0.242,"dns_end":36.614,"connect_start":36.614,"connect_end":80.609,"ssl_start":52.111,"ssl_end":80.6,"worker_start":-1,"worker_ready":-1,"worker_fetch_start":-1,"worker_respond_with_settled":-1,"send_start":80.808,"send_end":80.991,"push_start":0,"push_end":0,"receive_headers_end":195.368},"response_time":1609895099698.864,"protocol":"h2","security_state":"secure","security_details":{"protocol":"TLS 1.3","key_exchange":"","key_exchange_group":"X25519","cipher":"AES_128_GCM","certificate_id":0,"subject_name":"sni.cloudflaressl.com","san_list":["sni.cloudflaressl.com","*.vigneshh.in","vigneshh.in"],"issuer":"Cloudflare Inc ECC CA-3","valid_from":1595980800,"valid_to":1627560000,"signed_certificate_timestamp_list":[],"certificate_transparency_compliance":"unknown"}},"timings":{"blocked":0.24199998006224632,"queueing":1.9919999176636338,"proxy":-1,"dns":36.37200000230223,"ssl":28.488999931141734,"connect":43.99500007275492,"send":0.1829999964684248,"wait":114.37699990347028,"receive":1.8300000810995698,"total":199.1899999557063}},"package_version":"0.0.1-alpha.8"}

Verification

Timings for http://vigneshh.in -> NO SSL timing as its HTTP endpoint

{
  "blocked": 0.3469999646767974,
  "queueing": 1.1060000397264957,
  "proxy": -1,
  "dns": 34.92700005881488,
  "ssl": -1,
  "connect": 15.72399993892759,
  "send": 0.0849999487400055,
  "wait": 34.18199997395277,
  "receive": 1.6710000345483422,
  "total": 88.14600005280226
}

Timings for https://vigneshh.in

{
  "blocked": 0.24199998006224632,
  "queueing": 1.9919999176636338,
  "proxy": -1,
  "dns": 36.37200000230223,
  "ssl": 28.488999931141734,
  "connect": 43.99500007275492,
  "send": 0.1829999964684248,
  "wait": 114.37699990347028,
  "receive": 1.8300000810995698,
  "total": 199.1899999557063
}

@apmmachine
Copy link
Collaborator

apmmachine commented Jan 6, 2021

💔 Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Started by user Vignesh Shanmugam

  • Start Time: 2021-01-12T00:06:21.063+0000

  • Duration: 13 min 56 sec

Test stats 🧪

Test Results
Failed 0
Passed 47
Skipped 0
Total 47

Steps errors 2

Expand to view the steps failures

Build Docker image
  • Took 0 min 0 sec . View more details on here
Build Docker image
  • Took 0 min 0 sec . View more details on here

Log output

Expand to view the last 100 lines of log output

[2021-01-12T00:19:11.336Z] npm notice 1.6kB  dist/core/index.js.map               
[2021-01-12T00:19:11.336Z] npm notice 178B   dist/dsl/index.js.map                
[2021-01-12T00:19:11.336Z] npm notice 1.5kB  dist/index.js.map                    
[2021-01-12T00:19:11.336Z] npm notice 221B   dist/plugins/index.js.map            
[2021-01-12T00:19:11.336Z] npm notice 252B   dist/reporters/index.js.map          
[2021-01-12T00:19:11.336Z] npm notice 1.0kB  dist/dsl/journey.d.ts.map            
[2021-01-12T00:19:11.336Z] npm notice 776B   dist/dsl/journey.js.map              
[2021-01-12T00:19:11.336Z] npm notice 659B   dist/reporters/json.d.ts.map         
[2021-01-12T00:19:11.336Z] npm notice 3.0kB  dist/reporters/json.js.map           
[2021-01-12T00:19:11.336Z] npm notice 225B   dist/core/logger.d.ts.map            
[2021-01-12T00:19:11.336Z] npm notice 925B   dist/core/logger.js.map              
[2021-01-12T00:19:11.336Z] npm notice 827B   dist/plugins/network.d.ts.map        
[2021-01-12T00:19:11.336Z] npm notice 7.1kB  dist/plugins/network.js.map          
[2021-01-12T00:19:11.336Z] npm notice 221B   dist/parse_args.d.ts.map             
[2021-01-12T00:19:11.336Z] npm notice 1.2kB  dist/parse_args.js.map               
[2021-01-12T00:19:11.336Z] npm notice 757B   dist/plugins/performance.d.ts.map    
[2021-01-12T00:19:11.336Z] npm notice 1.0kB  dist/plugins/performance.js.map      
[2021-01-12T00:19:11.337Z] npm notice 967B   dist/plugins/plugin-manager.d.ts.map 
[2021-01-12T00:19:11.337Z] npm notice 1.9kB  dist/plugins/plugin-manager.js.map   
[2021-01-12T00:19:11.337Z] npm notice 3.2kB  dist/core/runner.d.ts.map            
[2021-01-12T00:19:11.337Z] npm notice 6.6kB  dist/core/runner.js.map              
[2021-01-12T00:19:11.337Z] npm notice 342B   dist/dsl/step.d.ts.map               
[2021-01-12T00:19:11.337Z] npm notice 328B   dist/dsl/step.js.map                 
[2021-01-12T00:19:11.337Z] npm notice 687B   dist/plugins/tracing.d.ts.map        
[2021-01-12T00:19:11.337Z] npm notice 1.5kB  dist/plugins/tracing.js.map          
[2021-01-12T00:19:11.337Z] npm notice 1.7kB  README.md                            
[2021-01-12T00:19:11.337Z] npm notice 1.6kB  dist/reporters/base.d.ts             
[2021-01-12T00:19:11.337Z] npm notice 4.3kB  src/reporters/base.ts                
[2021-01-12T00:19:11.337Z] npm notice 1.6kB  dist/plugins/browser-console.d.ts    
[2021-01-12T00:19:11.337Z] npm notice 2.2kB  src/plugins/browser-console.ts       
[2021-01-12T00:19:11.337Z] npm notice 1.2kB  dist/browser-service.d.ts            
[2021-01-12T00:19:11.337Z] npm notice 2.0kB  src/browser-service.ts               
[2021-01-12T00:19:11.337Z] npm notice 1.2kB  dist/cli.d.ts                        
[2021-01-12T00:19:11.337Z] npm notice 5.0kB  src/cli.ts                           
[2021-01-12T00:19:11.337Z] npm notice 2.7kB  dist/common_types.d.ts               
[2021-01-12T00:19:11.337Z] npm notice 2.5kB  src/common_types.ts                  
[2021-01-12T00:19:11.337Z] npm notice 2.1kB  dist/core/gatherer.d.ts              
[2021-01-12T00:19:11.337Z] npm notice 3.1kB  src/core/gatherer.ts                 
[2021-01-12T00:19:11.337Z] npm notice 3.8kB  dist/helpers.d.ts                    
[2021-01-12T00:19:11.337Z] npm notice 6.6kB  src/helpers.ts                       
[2021-01-12T00:19:11.337Z] npm notice 1.8kB  dist/core/index.d.ts                 
[2021-01-12T00:19:11.337Z] npm notice 1.2kB  dist/dsl/index.d.ts                  
[2021-01-12T00:19:11.337Z] npm notice 1.4kB  dist/index.d.ts                      
[2021-01-12T00:19:11.337Z] npm notice 1.3kB  dist/plugins/index.d.ts              
[2021-01-12T00:19:11.337Z] npm notice 1.4kB  dist/reporters/index.d.ts            
[2021-01-12T00:19:11.337Z] npm notice 2.8kB  src/core/index.ts                    
[2021-01-12T00:19:11.337Z] npm notice 1.2kB  src/dsl/index.ts                     
[2021-01-12T00:19:11.337Z] npm notice 2.7kB  src/index.ts                         
[2021-01-12T00:19:11.337Z] npm notice 1.3kB  src/plugins/index.ts                 
[2021-01-12T00:19:11.337Z] npm notice 1.3kB  src/reporters/index.ts               
[2021-01-12T00:19:11.337Z] npm notice 2.1kB  dist/dsl/journey.d.ts                
[2021-01-12T00:19:11.337Z] npm notice 2.2kB  src/dsl/journey.ts                   
[2021-01-12T00:19:11.337Z] npm notice 1.6kB  dist/reporters/json.d.ts             
[2021-01-12T00:19:11.337Z] npm notice 5.0kB  src/reporters/json.ts                
[2021-01-12T00:19:11.337Z] npm notice 1.3kB  dist/core/logger.d.ts                
[2021-01-12T00:19:11.337Z] npm notice 1.8kB  src/core/logger.ts                   
[2021-01-12T00:19:11.337Z] npm notice 2.7kB  dist/plugins/network.d.ts            
[2021-01-12T00:19:11.337Z] npm notice 9.7kB  src/plugins/network.ts               
[2021-01-12T00:19:11.337Z] npm notice 1.3kB  dist/parse_args.d.ts                 
[2021-01-12T00:19:11.337Z] npm notice 3.0kB  src/parse_args.ts                    
[2021-01-12T00:19:11.337Z] npm notice 1.8kB  dist/plugins/performance.d.ts        
[2021-01-12T00:19:11.337Z] npm notice 2.4kB  src/plugins/performance.ts           
[2021-01-12T00:19:11.337Z] npm notice 2.1kB  dist/plugins/plugin-manager.d.ts     
[2021-01-12T00:19:11.337Z] npm notice 3.4kB  src/plugins/plugin-manager.ts        
[2021-01-12T00:19:11.337Z] npm notice 4.4kB  dist/core/runner.d.ts                
[2021-01-12T00:19:11.337Z] npm notice 9.6kB  src/core/runner.ts                   
[2021-01-12T00:19:11.337Z] npm notice 1.4kB  dist/dsl/step.d.ts                   
[2021-01-12T00:19:11.337Z] npm notice 1.4kB  src/dsl/step.ts                      
[2021-01-12T00:19:11.337Z] npm notice 2.1kB  dist/plugins/tracing.d.ts            
[2021-01-12T00:19:11.337Z] npm notice 3.4kB  src/plugins/tracing.ts               
[2021-01-12T00:19:11.337Z] npm notice 71.6kB dist/tsconfig.tsbuildinfo            
[2021-01-12T00:19:11.337Z] npm notice === Tarball Details === 
[2021-01-12T00:19:11.337Z] npm notice name:          @elastic/synthetics                     
[2021-01-12T00:19:11.337Z] npm notice version:       0.0.1-alpha.8                           
[2021-01-12T00:19:11.337Z] npm notice filename:      elastic-synthetics-0.0.1-alpha.8.tgz    
[2021-01-12T00:19:11.337Z] npm notice package size:  63.4 kB                                 
[2021-01-12T00:19:11.337Z] npm notice unpacked size: 335.4 kB                                
[2021-01-12T00:19:11.337Z] npm notice shasum:        50a8231514d06c374b05aada5fca9b3273820895
[2021-01-12T00:19:11.337Z] npm notice integrity:     sha512-u7oe6p/cxQwbE[...]1YMH2GK6aJhVQ==
[2021-01-12T00:19:11.337Z] npm notice total files:   114                                     
[2021-01-12T00:19:11.337Z] npm notice 
[2021-01-12T00:19:11.337Z] elastic-synthetics-0.0.1-alpha.8.tgz
[2021-01-12T00:19:13.078Z] Running in /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-180/src/github.com/elastic/synthetics
[2021-01-12T00:19:13.096Z] [INFO] getVaultSecret: Getting secrets
[2021-01-12T00:19:13.436Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2021-01-12T00:19:14.151Z] + set +x
[2021-01-12T00:19:14.413Z] Login Succeeded
[2021-01-12T00:19:14.767Z] + date +%Y-%m-%d_%H:%M
[2021-01-12T00:19:14.767Z] + docker build -t docker.elastic.co/observability-ci/synthetics:f55697f54d16332231cab9d6818c2f12e1534745-7.10.0-synthetics -t docker.elastic.co/observability-ci/synthetics:PR-180-7.10.0-synthetics --build-arg STACK_VERSION=7.10.0-synthetics --label BRANCH_NAME=PR-180 --label GIT_SHA=f55697f54d16332231cab9d6818c2f12e1534745 --label TIMESTAMP=2021-01-12_00:19 .
[2021-01-12T00:19:14.767Z] Sending build context to Docker daemon  3.057MB

[2021-01-12T00:19:14.767Z] Error response from daemon: dockerfile parse error line 22: unknown instruction: /USR/SHARE/HEARTBEAT/.NPM
[2021-01-12T00:19:15.708Z] Stage "experimental" skipped due to earlier failure(s)
[2021-01-12T00:19:15.802Z] Failed in branch Matrix - STACK_VERSION = '7.10.0-synthetics'
[2021-01-12T00:19:16.110Z] Running on Jenkins in /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-180
[2021-01-12T00:19:16.151Z] [INFO] getVaultSecret: Getting secrets
[2021-01-12T00:19:16.498Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2021-01-12T00:19:17.118Z] + chmod 755 generate-build-data.sh
[2021-01-12T00:19:17.118Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-180/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-180/runs/3 FAILURE 775796
[2021-01-12T00:19:17.668Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-180/runs/3/steps/?limit=10000 -o steps-info.json
[2021-01-12T00:19:18.219Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-180/runs/3/tests/?status=FAILED -o tests-errors.json

Copy link
Contributor

@andrewvc andrewvc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Found a potential issue. Also, can we add a unit test around this logic?

src/plugins/network.ts Show resolved Hide resolved
Copy link
Contributor

@andrewvc andrewvc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vigneshshanmugam
Copy link
Member Author

Docker build issue is fixed by #179

@vigneshshanmugam vigneshshanmugam merged commit b55ab0a into elastic:master Jan 12, 2021
@vigneshshanmugam vigneshshanmugam deleted the fix-redirectss branch January 12, 2021 00:25
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.

Redirects and DNS failures not represented correctly
3 participants