Wordpress - Preview Post - REQUEST-941-APPLICATION-ATTACK-XSS.conf

Receiving a false positive when trying to preview changes to a page within WordPress. Seems to be related to posts that have HTML markup within the post/content area. If I try to preview a page/post that is text only, there is no error.
--8033876e-H--
Message: Warning. detected XSS using libinjection. [file "/usr/share/modsecurity-crs/rules/REQUEST-941-APPLICATION-ATTACK-XSS.conf"] [line "60"] [id "941100"] [msg "XSS Attack Detected via libinjection"] [data "Matched Data: XSS data found within ARGS:acf[field_5d2deda542dca][row-0][field_5d2dedf9216aa]:
\x0d\x0a
\x0d\x0a\x0d\x0a<img class=\x22aligncenter wp-image-1930 size-medium\x22 title=\x22EdTech Awards Cool Tool Winner 2021\x22 src=\x22https://XXXXXXX/wp-content/uploads/2021/04/edtech_..."] [severity "CRITICAL"] [ver "OWASP_CRS/3.2.0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-xss"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "OWASP_CRS/WEB_ATTACK/XSS"] [tag "WASCTC/WASC-8"] [tag "WASCTC/WASC-22"] [tag "OWASP_TOP_10/A3"] [tag "OWASP_AppSensor/IE
Message: Warning. Pattern match "(?i:(?:<\\w[\\s\\S]*[\\s\\/]|['\"](?:[\\s\\S]*[\\s\\/])?)(?:on(?:d(?:e(?:vice(?:(?:orienta|mo)tion|proximity|found|light)|livery(?:success|error)|activate)|r(?:ag(?:e(?:n(?:ter|d)|xit)|(?:gestur|leav)e|start|drop|over)|op)|i(?:s(?:c(?:hargingtimechange ..." at ARGS:acf[field_5d2deda542dca][row-0][field_5d2dedf9216aa]. [file "/usr/share/modsecurity-crs/rules/REQUEST-941-APPLICATION-ATTACK-XSS.conf"] [line "229"] [id "941160"] [msg "NoScript XSS InjectionChecker: HTML Injection"] [data "Matched Data:
\x0d\x0a
\x0d\x0a\x0d\x0a<img class=\x22aligncenter wp-image-1930 size-medium\x22 title=\x22EdTech Awards Cool Tool Winner 2021\x22 src=\x22https://XXXXXXX/wp-content/uploads/2021/04/edtech_winner-250x373.png\x22 alt=\x22EdTech Awards Cool Tool Winner 2021\x22 width=\x22
Message: Rule 7f45041e1330 [id "941200"][file "/usr/share/modsecurity-crs/rules/REQUEST-941-APPLICATION-ATTACK-XSS.conf"][line "349"] - Execution error - PCRE limits exceeded (-8): (null).
Message: Rule 7f450416eea0 [id "941350"][file "/usr/share/modsecurity-crs/rules/REQUEST-941-APPLICATION-ATTACK-XSS.conf"][line "683"] - Execution error - PCRE limits exceeded (-8): (null).
Message: Access denied with code 403 (phase 2). Operator GE matched 5 at TX:anomaly_score. [file "/usr/share/modsecurity-crs/rules/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "93"] [id "949110"] [msg "Inbound Anomaly Score Exceeded (Total Score: 10)"] [severity "CRITICAL"] [ver "OWASP_CRS/3.2.0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"]
Message: Warning. Operator GE matched 5 at TX:inbound_anomaly_score. [file "/usr/share/modsecurity-crs/rules/RESPONSE-980-CORRELATION.conf"] [line "91"] [id "980130"] [msg "Inbound Anomaly Score Exceeded (Total Inbound Score: 10 - SQLI=0,XSS=10,RFI=0,LFI=0,RCE=0,PHPI=0,HTTP=0,SESS=0): individual paranoia level scores: 10, 0, 0, 0"] [ver "OWASP_CRS/3.2.0"] [tag "event-correlation"]
Apache-Error: [file "apache2_util.c"] [line 273] [level 3] [client XXX.XXX.XXX.XXX] ModSecurity: Warning. detected XSS using libinjection. [file "/usr/share/modsecurity-crs/rules/REQUEST-941-APPLICATION-ATTACK-XSS.conf"] [line "60"] [id "941100"] [msg "XSS Attack Detected via libinjection"] [data "Matched Data: XSS data found within ARGS:acf[field_5d2deda542dca][row-0][field_5d2dedf9216aa]:
\\\\x0d\\\\x0a
\\\\x0d\\\\x0a\\\\x0d\\\\x0a<img class=\\\\x22aligncenter wp-image-1930 size-medium\\\\x22 title=\\\\x22EdTech Awards Cool Tool Winner 2021\\\\x22 src=\\\\x22https://XXXXXXX/wp-content/uploads/2021/04/edtech_..."] [severity "CRITICAL"] [ver "OWASP_CRS/3.2.0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-xss"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "OWASP_CRS/WEB_ATTACK/XSS"] [tag "WASCTC/WASC-8"] [tag "WASCTC/WASC-22"] [tag "OWASP_TOP_10/A3"] [tag "OWASP_AppSensor/IE [hostname "XXXXXXX"] [uri "/wp-admin/post.php"] [unique_id "YJLE6UbdDCVJBG6bxGTwAwAAAA0"]
Apache-Error: [file "apache2_util.c"] [line 273] [level 3] [client XXX.XXX.XXX.XXX] ModSecurity: Warning. Pattern match "(?i:(?:<\\\\\\\\w[\\\\\\\\s\\\\\\\\S]*[\\\\\\\\s\\\\\\\\/]|['\\\\"](?:[\\\\\\\\s\\\\\\\\S]*[\\\\\\\\s\\\\\\\\/])?)(?:on(?:d(?:e(?:vice(?:(?:orienta|mo)tion|proximity|found|light)|livery(?:success|error)|activate)|r(?:ag(?:e(?:n(?:ter|d)|xit)|(?:gestur|leav)e|start|drop|over)|op)|i(?:s(?:c(?:hargingtimechange ..." at ARGS:acf[field_5d2deda542dca][row-0][field_5d2dedf9216aa]. [file "/usr/share/modsecurity-crs/rules/REQUEST-941-APPLICATION-ATTACK-XSS.conf"] [line "229"] [id "941160"] [msg "NoScript XSS InjectionChecker: HTML Injection"] [data "Matched Data:
\\\\x0d\\\\x0a
\\\\x0d\\\\x0a\\\\x0d\\\\x0a<img class=\\\\x22aligncenter wp-image-1930 size-medium\\\\x22 title=\\\\x22EdTech Awards Cool Tool Winner 2021\\\\x22 src=\\\\x22https://XXXXXXX/wp-content/uploads/2021/04/edtech_winner-250x373.png\\\\x22 alt=\\\\x22EdTech Awards Cool Tool Winner 2021\\\\x22 width=\\\\x22 [hostname "XXXXXXX"] [uri "/wp-admin/post.php"] [unique_id "YJLE6UbdDCVJBG6bxGTwAwAAAA0"]
Apache-Error: [file "apache2_util.c"] [line 273] [level 3] [client XXX.XXX.XXX.XXX] ModSecurity: Rule 7f45041e1330 [id "941200"][file "/usr/share/modsecurity-crs/rules/REQUEST-941-APPLICATION-ATTACK-XSS.conf"][line "349"] - Execution error - PCRE limits exceeded (-8): (null). [hostname "XXXXXXX"] [uri "/wp-admin/post.php"] [unique_id "YJLE6UbdDCVJBG6bxGTwAwAAAA0"]
Apache-Error: [file "apache2_util.c"] [line 273] [level 3] [client XXX.XXX.XXX.XXX] ModSecurity: Rule 7f450416eea0 [id "941350"][file "/usr/share/modsecurity-crs/rules/REQUEST-941-APPLICATION-ATTACK-XSS.conf"][line "683"] - Execution error - PCRE limits exceeded (-8): (null). [hostname "XXXXXXX"] [uri "/wp-admin/post.php"] [unique_id "YJLE6UbdDCVJBG6bxGTwAwAAAA0"]
Apache-Error: [file "apache2_util.c"] [line 273] [level 3] [client XXX.XXX.XXX.XXX] ModSecurity: Access denied with code 403 (phase 2). Operator GE matched 5 at TX:anomaly_score. [file "/usr/share/modsecurity-crs/rules/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "93"] [id "949110"] [msg "Inbound Anomaly Score Exceeded (Total Score: 10)"] [severity "CRITICAL"] [ver "OWASP_CRS/3.2.0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"] [hostname "XXXXXXX"] [uri "/wp-admin/post.php"] [unique_id "YJLE6UbdDCVJBG6bxGTwAwAAAA0"]
Apache-Error: [file "apache2_util.c"] [line 273] [level 3] [client XXX.XXX.XXX.XXX] ModSecurity: Warning. Operator GE matched 5 at TX:inbound_anomaly_score. [file "/usr/share/modsecurity-crs/rules/RESPONSE-980-CORRELATION.conf"] [line "91"] [id "980130"] [msg "Inbound Anomaly Score Exceeded (Total Inbound Score: 10 - SQLI=0,XSS=10,RFI=0,LFI=0,RCE=0,PHPI=0,HTTP=0,SESS=0): individual paranoia level scores: 10, 0, 0, 0"] [ver "OWASP_CRS/3.2.0"] [tag "event-correlation"] [hostname "XXXXXXX"] [uri "/wp-admin/post.php"] [unique_id "YJLE6UbdDCVJBG6bxGTwAwAAAA0"]
Action: Intercepted (phase 2)
Apache-Handler: application/x-httpd-php
Stopwatch:<PHONE_NUMBER>261735 137309 (- - -)
Stopwatch2:<PHONE_NUMBER>261735 137309; combined=136005, p1=422, p2=135378, p3=0, p4=0, p5=205, sr=75, sw=0, l=0, gc=0
Response-Body-Transformed: Dechunked
Producer: ModSecurity for Apache/2.9.3 (http://www.modsecurity.org/); OWASP_CRS/3.2.0.
Server: Apache/2.4.38 (Debian) OpenSSL/1.1.1d
Engine-Mode: "ENABLED"

CRS version v3.2.0
Paranoia level setting: 1
ModSecurity version 2.9.3)
Apache 2.4.38
Debian Buster
Wordpress exclusions enabled

This is a problem with the Advanced Custom Fields (ACF) plugin sadly. This plugin sends parameters named ARGS:acf[field_5d2deda542dca][row-0][field_5d2dedf9216aa] like in your example. These parameters have  constantly changing names.
Their names do follow a standard pattern, but unfortunately, we can't use regular expressions in argument names when making exclusions. The only option is to exclude all of ARGS. And I didn't want to make that a part of the standard WordPress exclusion package, since it would open up the firewall too much.
If you run into this problem and you want to keep using ACF with the CRS, then the best option is to write a custom exception rule for it that allows all ARGS when saving a post. You didn't share the whole body section of your post, so I can't give a rule now, but if you need help writing such a rule then please re-open this message.

Thanks for the response. Could definitely use the help creating the expression. Here is the full dump...
--8033876e-A--
[05/May/2021:11:16:41 --0500] YJLE6UbdDCVJBG6bxGTwAwAAAA0 XXX.XXX.XXX.XXX 24898 XXX.XXX.XXX.XXX 443
--8033876e-B--
POST /wp-admin/post.php HTTP/1.1
Host: XXXXXXX
Connection: Keep-Alive
Accept-Encoding: gzip
CF-IPCountry: US
X-Forwarded-For: XXX.XXX.XXX.XXX
CF-RAY: 64ab46513dec4959-STL
Content-Length: 4344
X-Forwarded-Proto: https
CF-Visitor: {"scheme":"https"}
user-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:88.0) Gecko/20100101 Firefox/88.0
accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,*/*;q=0.8
accept-language: en-US,en;q=0.5
referer: https://XXXXXXX/wp-admin/post.php?post=1923&action=edit
content-type: application/x-www-form-urlencoded
origin: https://XXXXXXX
dnt: 1
upgrade-insecure-requests: 1
pragma: no-cache
cache-control: no-cache
cookie: wordpress_sec_596f5d061128c5c12adf99623f04534c=XXXXXX%7C1620400720%7Cbca2hnabVx4HyIUfxUjnMM3Tx74wYeKnJqfg9shSoaw%7Cf25a0bceb1146504c53606c46b39d2414389c7659bfafcc0e37af9e88f615873; wp-saving-post=1923-check; tatari-session-cookie=5c23c803-189c-613f-0e81-369afec657bc; __zlcmid=s3iCQ2tBkihDIH; _derived_epik=dj0yJnU9UHZyUHo0ZXFnZEZYNmt5aTAtNWJPTkh4dExjMUI1NnYmbj1XMlR0R2FmT2VvWTZoLWNVV2Ftd01BJm09MSZ0PUFBQUFBR0JuaXhzJnJtPTEmcnQ9QUFBQUFHQm5peHM; _ga=GA1.2.amp-uJNtzbj7ZtrikQQ34cYguQ; _pin_unauth=dWlkPVpXTTJNalUxTmpBdFpUTXpaaTAwTkRjd0xXSmtNelF0WVRObFpqZzRPVGM1WlRZMQ; _gcl_au=1.1.419792397.1597248535; geot_country=US; _gaexp=GAX1.2.bNAaDvLyT-qKHP3uSAEryw.18755.0; wp-settings-2=editor%3Dtinymce%26libraryContent%3Dbrowse%26posts_list_mode%3Dlist%26advImgDetails%3Dshow; wp-settings-time-2=1618584404; __cfduid=df74b08fdededfcec9284a033285f1ce81619196181; PHPSESSID=3dsb4bjq38arunffuu7rbtqgb8; geot_rocket_country=US; geot_rocket_state=not-detected; geot_rocket_city=not-detected; login_referer=https%3A%2F%2FXXXXXXX%2Fwp-admin%2F; wordpress_test_cookie=WP+Cookie+check; wordpress_logged_in_596f5d061128c5c12adf99623f04534c=XXXXXX%7C1620400720%7Cbca2hnabVx4HyIUfxUjnMM3Tx74wYeKnJqfg9shSoaw%7C4295403c8f12d806b877f51c2d793d1ac461cb8fe15d6268bd96b3b927afa469
CF-Connecting-IP: XXX.XXX.XXX.XXX
CDN-Loop: cloudflare
cf-request-id: 09deea46c6000049594d3a1000000001
GEOIP_ADDR: XXX.XXX.XXX.XXX
GEOIP_CONTINENT_CODE: NA
GEOIP_COUNTRY_CODE: US
GEOIP_COUNTRY_NAME: United States

--8033876e-C--
_wpnonce=94b3b040b5&_wp_http_referer=%2Fwp-admin%2Fpost.php%3Fpost%3D1923%26action%3Dedit&user_ID=2&action=editpost&originalaction=editpost&post_author=2&post_type=page&original_post_status=publish&referredby=https%3A%2F%2FXXXXXXX%2Fwp-admin%2Fedit.php%3Fpost_type%3Dpage&_wp_original_http_referer=https%3A%2F%2FXXXXXXX%2Fwp-admin%2Fedit.php%3Fpost_type%3Dpage&post_ID=1923&meta-box-order-nonce=2044ba7dd6&closedpostboxesnonce=cb6a84ad48&post_title=2021+EdTech+Awards&samplepermalinknonce=c6ce2175a5&_acf_screen=post&_acf_post_id=1923&_acf_validation=1&_acf_nonce=51759deb6f&_acf_changed=0&content=Osmo+is+happy+to+announce+winning+awards+for+the+2021+EdTech+Digest+Award+for+Curriculum+and+Instruction+with+the+Osmo+Learning+Systems+and+Finalists+New+Product+with+the+Projector+and+the+STEM+for+the+STEM+Solution%21&wp-preview=dopreview&hidden_post_status=publish&post_status=publish&hidden_post_password=&hidden_post_visibility=public&visibility=public&post_password=&mm=04&jj=16&aa=2021&hh=14&mn=39&ss=16&hidden_mm=04&cur_mm=05&hidden_jj=16&cur_jj=05&hidden_aa=2021&cur_aa=2021&hidden_hh=14&cur_hh=15&hidden_mn=39&cur_mn=40&original_publish=Update&tax_input%5Bpost_tag%5D=&newtag%5Bpost_tag%5D=&parent_id=&page_template=template-full-skinny.php&menu_order=0&tml_restrictions_meta_box=9ed4ed8f48&_wp_http_referer=%2Fwp-admin%2Fpost.php%3Fpost%3D1923%26action%3Dedit&_thumbnail_id=-1&yoast_free_metabox_nonce=7da41f690e&_wp_http_referer=%2Fwp-admin%2Fpost.php%3Fpost%3D1923%26action%3Dedit&yoast_wpseo_focuskw=&yoast_wpseo_title=&yoast_wpseo_metadesc=&yoast_wpseo_linkdex=-1393&yoast_wpseo_content_score=30&yoast_wpseo_is_cornerstone=false&yoast_wpseo_estimated-reading-time-minutes=1&yoast_wpseo_meta-robots-noindex=0&yoast_wpseo_meta-robots-nofollow=0&yoast_wpseo_meta-robots-adv=&yoast_wpseo_canonical=&yoast_wpseo_schema_page_type=&yoast_wpseo_opengraph-title=&yoast_wpseo_opengraph-description=&yoast_wpseo_opengraph-image=&yoast_wpseo_opengraph-image-id=&yoast_wpseo_twitter-title=&yoast_wpseo_twitter-description=&yoast_wpseo_twitter-image=&yoast_wpseo_twitter-image-id=&screen=mobile&acf%5Bfield_5e7e108dbf0bf%5D=0&acf%5Bfield_5e7e302a6b29a%5D=&acf%5Bfield_5e7e4069b328d%5D=&acf%5Bfield_5d2deda542dca%5D=&acf%5Bfield_5d2deda542dca%5D%5Brow-0%5D%5Bacf_fc_layout%5D=flexible-content&acf%5Bfield_5d2deda542dca%5D%5Brow-0%5D%5Bfield_5d2dedf9216aa%5D=%3Cdiv+class%3D%22grid-row%22%3E%0D%0A%3Cdiv+class%3D%22col-xs-12+col-md-6%22+style%3D%22text-align%3A+center%3B%22%3E%0D%0A%0D%0A%3Ca+href%3D%22https%3A%2F%2Fwww.edtechdigest.com%2F2021-finalists-winners%2F%22+target%3D%22_blank%22+rel%3D%22noopener%22%3E%3Cimg+class%3D%22aligncenter+wp-image-1930+size-medium%22+title%3D%22EdTech+Awards+Cool+Tool+Winner+2021%22+src%3D%22https%3A%2F%2FXXXXXXX%2Fwp-content%2Fuploads%2F2021%2F04%2Fedtech_winner-250x373.png%22+alt%3D%22EdTech+Awards+Cool+Tool+Winner+2021%22+width%3D%22201%22+height%3D%22300%22+%2F%3E%3C%2Fa%3E%0D%0A%3Ch2%3ECurriculum+and+Instruction%3C%2Fh2%3E%0D%0AOsmo+Learning+Systems%0D%0A%0D%0A%3C%2Fdiv%3E%0D%0A%3Cdiv+class%3D%22col-xs-12+col-md-6%22+style%3D%22text-align%3A+center%3B%22%3E%0D%0A%0D%0A%3Ca+href%3D%22https%3A%2F%2Fwww.edtechdigest.com%2F2021-finalists-winners%2F%22+target%3D%22_blank%22+rel%3D%22noopener%22%3E%3Cimg+class%3D%22aligncenter+wp-image-1931+size-medium%22+title%3D%22EdTech+Awards+Cool+Tool+Finalist+2021%22+src%3D%22https%3A%2F%2FXXXXXXX%2Fwp-content%2Fuploads%2F2021%2F04%2Fedtech_finalist-250x373.png%22+alt%3D%22EdTech+Awards+Cool+Tool+Finalist+2021%22+width%3D%22201%22+height%3D%22300%22+%2F%3E%3C%2Fa%3E%0D%0A%3Ch2%3ENew+Product%3C%2Fh2%3E%0D%0AOsmo+Projector%0D%0A%3Ch2%3ESTEM+Solution%3C%2Fh2%3E%0D%0AOsmo+Learning+System+STEAM+Edition%0D%0A%0D%0A%3C%2Fdiv%3E%0D%0A%3C%2Fdiv%3E&advanced_view=1&add_comment_nonce=a68e880af6&_ajax_fetch_list_nonce=6cb0cfcc9b&_wp_http_referer=%2Fwp-admin%2Fpost.php%3Fpost%3D1923%26action%3Dedit&post_name=2021-edtech-awards&post_author_override=2&geot%5Bgeot_include_mode%5D=include&geot%5Bcities%5D=&geot%5Bstates%5D=&geot%5Bzipcodes%5D=&geot%5Bradius_km%5D=&geot%5Bradius_lat%5D=&geot%5Bradius_lng%5D=&geot%5Bforbidden_text%5D=This+content+is+restricted+in+your+region&geot_options_nonce=3d001cee66&_wp_http_referer=%2Fwp-admin%2Fpost.php%3Fpost%3D1923%26action%3Dedit&heateor_sss_meta_nonce=bad785f871
--8033876e-F--
HTTP/1.1 403 Forbidden
Content-Length: 199
Keep-Alive: timeout=5, max=100
Connection: Keep-Alive
Content-Type: text/html; charset=iso-8859-1

--8033876e-E--
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>403 Forbidden</title>

Forbidden

You don't have permission to access this resource.

What I've used in the past was an approach like this:
Create/ddd to the REQUEST-900-EXCLUSION-RULES-BEFORE-CRS.conf:
SecRuleUpdateTargetById 941100 "!ARGS_NAMES:/^acf\[field_ ... (add a regexp that match your field)/"

That way you don't need to remove all ARGS.

For the record: @fzipi's proposal works nicely, but it removes the parameter(s) for the whole server and that's a practice we can not / do not want to do with the rule exclusion package. But if you know your server, then there is nothing wrong with using that recipe.


Proposal: SpawnPlugin

A Plugin that allows running spawn before webpack starting the compilation and after it's done.
This will allow a lot of freedom but for me the main reason is to simplify the work with npm scripts. Since a lot of tasks do the same thing with minor changes you get a long list of scripts referencing other scripts with a changed flag or arg.
Consider the UglifyJS, Copy etc... Webpack plugins, they can all run from the cli.
In these cases a plugin is appropriate since CLI is complicated and sometimes you need to change the tree but there are a lot of cases that a plugin isn't really needed just a simple command line hooks will do the job.
API
The first release should be simple, we can extend the use cases later.
new SpawnPlugin({
  before: [
     'myScript --param1 value --param2 value'
  ],
  after: [
      'myScript --param1 value --param2 value'
  ]
})

This is the default implementation, the plugin with convert the string into a command and params array and invoke a spawn attached to the console.
export class SimplePlugin {

  constructor(options: SimplePluginOptions) {
  }

  apply(compiler: any): void {
    compiler.plugin('run', (compiler, next) => this.run(next) );
    compiler.plugin('watch-run', (compiler, next) => this.run(next) );
    compiler.plugin('done', (stats) => this.done(stats) );
  }

  run(next: (err?: Error) => any): void {
    // for each BEFORE script.
    spawn('..'}
      .catch( err => err) )
      .then( err => next(err as any) );
  }

  done(stats: any): void {
      // for each AFTER script.
      spawnSync(`..`);
  }
}

This is TS code...
spawn and spawnSync are wrappers around node spawn/spawnSync that returns a promise and also parse the string into a command and args.
It also implements {stdio: "inherit"} by default so all communication goes to the console.
If this is approved, the next versions can include more control over spawn but supplying an object instead of a string that can change the default behaviour. An object like this is actually a configuration per spawn object... but again after first release and if approved.

I approved i'll post a PR

Feel free to publish the plugin as separate plugin.

@sokra Makes sense.
Where can I find the current tags? do you have a link?


artisan make:auth on laravel 5.3.22

Laravel Version: 5.3.22
PHP Version: 5.6.15
Database Driver & Version: mariaDB 10.1.8-MariaDB

Description:
the make:auth artisan command produces a broken login controller, the system can manage register just fine but when trying to log in, it impossible, as you can see in this SO Post i have done some troubleshooting to no avail, also I tested the same steeps with Laravel 5.2 and it worked right
Steps To Reproduce:

composer create-project --prefer-dist laravel/laravel blog
cd blog
edit .env to set the database conection
artisan make:auth
artisan migrate
artisan serve
register a new user using the generated /register page
log out
try to log in
log in will fail with a These credentials do not match our records. error

Laravel 5.3.22
PHP 5.6.15
MariaDB 10.1.9

Couldn't reproduce this issue.
I had a TokenMismatchException when tried this with php artisan serve but when tested on a classic Apache server it worked fine.
Do you have an entry in your database for the new user? Does the hash match the password?
If you haven't checked, try placing this:
Artisan::command('test', function () {
    dd(Hash::check('secret', \App\User::first()->password));
});

in routes/console.php file and run php artisan test.
Did you try authenticating manually?
public function index(\Illuminate\Contracts\Auth\Guard $auth, \Illuminate\Http\Request $request)
{
    if($auth->attempt(['email' => $request->email, 'password' => $request->password])) {
        return 'Yup';
    }

    return 'Nope';
}

how strange, i just tested again and it worked, first i thought that it was because i was using power shell instead of cmd, but tested it on cmd and it also worked, so i guess is a moon phase bug, anyway i'll make a unit test and come back if ti repeats again

Probably you just typed the wrong password :)
Besides, PowerShell FTW!

hi it, just happened again, i have started to think the directory is haunted or something
the only change is that in this moment im conected via a cellsphone hotspot,, il try again at my home to rule this out

i figured out the problem, i'm an idiot, in the migrations i wrote "Password" instead of "password" (Capital P), sorry for wasting your time


WIP - SAM mega study details

Closes #696
Closes #275
This PR is still a WIP. It aims to add a panel for sub-studies visible on the map, and to retain panel position and dimensions.
TODO:

[ ] Refine verbiage
[ ] Add new header component to all map types
[ ] Only show new header component in mega studies. This may need to be hard coded (as a top-level prop) for now.
[ ] Make sure the correct filters are being passed.
[ ] Figure out app state backwards compatibility (studyDetailsPanelConfig is a mandatory prop at the moment)

Screenshot of current progress

I've wire up bar and bubble markers, too. There is an inconsistency with bubble markers.
Oh, I see... the "denominator" is acting like a little filter. Hopefully this isn't too confusing.

I've hard coded the study entity and variable IDs for now. We can figure out how to make that configurable later.


nice

wow that is very sneaky! is there a way to do this on  linux or mac?

Umm not to my knowledge, something similar might be viable on OSX but probably not Linux.


feat: New prover documentation

New prover documentation.
Rendered form is recommended, as it's easier to view.
Additionally, does the following:

Introduces ordering for advanced guides (it was a mix, not it's more or less sorted and suggests an order for reading).
Fixes a few things in the setup-dev.md.
Creates a prover-local config profile, so that no manual interactions with configs are needed.
Gets rid of prover/setup.sh as it's no longer needed that much.

I'm not very familiar with the prover subsystem, but I just want to give a heads up to ensure you are aware that, ideally, we probably want to be able to optionally enable or disable the SGX prover (or other types of TEE provers) alongside the regular one. Not sure if that's something to be concerned about at this point, though. cc @haraldh


Moving an item to index 0 gets inserted to index 1

Try dragging an item from any position other than the top, up to the top (index 0) of a list
expected: Item gets inserted at the very top of the list, pushing other items down
actual: items gets inserted at the 2nd position (index 1)
It's weirdly inconsistent, I can get it to work in a nested list sometimes
This can be seen happening right now on the demo linked to from your Readme: https://stackblitz.com/edit/ngx-drag-and-drop-lists/?file=src%2Fapp%2Fapp.component.ts

I think I know why it's happening, there's an issue with the calculation of this offset value:  https://github.com/misha130/ngx-drag-and-drop-lists/blob/master/src/directives/dnd-list.ts#L162
let offset: number = this.nativeElement.children.length - 1 - this.dndModel.length;

The problem is that this.nativeElement.children includes a placeholder element which causes a classic off-by-1 issue

You probably need to filter out the placeholder element when checking children
So right now, when trying to drag to position 0, the offset value gets calculated to a value of -1
And when we get to this line: https://github.com/misha130/ngx-drag-and-drop-lists/blob/master/src/directives/dnd-list.ts#L162
let insertionPoint: number = index - offset;

index 0 minus offset -1 = 1(0 - -1 = 1)
And that's why anything trying to get inserted into index 0 keeps ended up at index 1

For anyone else experiencing this issue I have a temporary workaround using the dndInserted event to alter the model after inserted.
// (dndInserted)="onInsert($event, simpleList[0])"

public onInsert(event: DropEvent<any>, list: any[]) {
    const actualIndex = list.indexOf(event.item);
    if (event.index !== actualIndex) {
      const [item] = list.splice(actualIndex, 1);
      list.splice(event.index, 0, item);
    }
  }

See it in action here (I did not include it in the nested list)
https://stackblitz.com/edit/ngx-drag-and-drop-lists-xbft7n/?file=src%2Fapp%2Fapp.component.html,src%2Fapp%2Fapp.component.ts


NOISSUE: fix bats tests

Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Pull Request title is properly formatted: [KOGITO|RHPAM-XYZ] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a testcase that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

How to retest this PR or trigger a specific build:

(Re)run Jenkins tests
Please add comment: Jenkins [test|retest] this

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

PR job #74 was: UNSTABLE
Possible explanation: This should be test failures
Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/74/display/redirect
See console log:

Console Logs
[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1738 - kogito-runtime-jvm: https://github.com/apache/incubator-kie-kogito-images/pull/1738 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1738 - kogito-jobs-service-ephemeral: https://github.com/apache/incubator-kie-kogito-images/pull/1738 completed: UNSTABLE[Pipeline] unstableWARNING: Tests on kogito-jobs-service-ephemeral seems to have failed[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1738 - kogito-jobs-service-allinone: https://github.com/apache/incubator-kie-kogito-images/pull/1738 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1738 - kogito-task-console: https://github.com/apache/incubator-kie-kogito-images/pull/1738 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1738 - kogito-management-console: https://github.com/apache/incubator-kie-kogito-images/pull/1738 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1738 - kogito-swf-builder: https://github.com/apache/incubator-kie-kogito-images/pull/1738 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1738 - kogito-swf-devmode: https://github.com/apache/incubator-kie-kogito-images/pull/1738 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » main » pullrequest » kogito-images.build-image PR #1738 - kogito-s2i-builder: https://github.com/apache/incubator-kie-kogito-images/pull/1738 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // parallel[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - 'https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-images.build-and-test/74/api/json?depth=0'[Pipeline] readJSON[Pipeline] sh


⚠ Bofrost.at has degraded performance

In a4ea91e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1813 ms

Resolved: Bofrost.at performance has improved in c210f69.


[Suggestion] Option to make Totem of the Soul work in inventory

Considering there are a lot of other mods to make the Totem of Undying work from anywhere in the inventory, it would be nice to have an option in Lacrimis to make its more expensive and powerful equivalent also work from the inventory, without requiring it be held all the time.

If there was a way that my mod could more easily be compatible with those ones I would do that, and as it stands it is possible for one of those mods to implement. The problem is that I certainly do not want my Totem working that way by default, so I would have to set up some config or a gamerule. Not sure if I will but I will add it to the list. If I do implement that it would also be easy to make the gamerule apply to both.


Proposal: align API with sklearn -> complete becomes fit_transform

We currently have complete, but this is identical in functionality to fit_transform. Also, with the changeover from MICE to IterativeImputer, we will have an imputer that supports an inductive fit/transform. In my opinion it is worth changing complete to fit_transform and figure out which of our other imputers can support an inductive fit/transform. Probably at least one!
@iskandr what do you think?

I think this makes sense. Every time I come back to fancyimpute, which is pretty infrequent, I've forgotten its bespoke API. Would be nice to align better with sklearn.

I'm on it, thanks.

This is done. The new version of fancyimpute is 0.4.0 and is now live: https://pypi.org/project/fancyimpute/


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit ed0e716:
:clock10: Pending: waiting for processors (31 builds ahead of you)
:warning: Docs Build is busy, currently there are 31 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit ed0e716:
:clock10: Full build: the average full build time is 10 min(s) 48 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit ed0e716:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


remove "characterization status" from header for ncRNAs

We don't curate this info for ncRNAs currently so we should probably not display it

That's fixed now.


Fetch calls return success for an unavailable provider

Once a provider has been loaded it does not seem possible to unload it again. Fetch calls still return algorithms from the unloaded provider - even though it is reported as unavailable from a call to OSSL_PROVIDER_available. See the following test code:
#include <stdio.h>
#include <openssl/evp.h>
#include <openssl/provider.h>

int main(void)
{
    OSSL_PROVIDER *legacy = OSSL_PROVIDER_load(NULL, "legacy");
    int ret = 1;
    EVP_MD *md4 = NULL;

    if (legacy == NULL) {
        printf("Failed to load legacy provider\n");
        goto end;
    }

    if (!OSSL_PROVIDER_available(NULL, "legacy")) {
        printf("Legacy provider should be available\n");
        goto end;
    }

    md4 = EVP_MD_fetch(NULL, "MD4", NULL);
    if (md4 == NULL) {
        printf("Failed in first fetch of md4\n");
        goto end;
    }
    EVP_MD_free(md4);

    OSSL_PROVIDER_unload(legacy);
    if (OSSL_PROVIDER_available(NULL, "legacy")) {
        printf("Legacy provider should NOT be available\n");
        goto end;
    }

    md4 = EVP_MD_fetch(NULL, "MD4", NULL);
    if (md4 != NULL) {
        const char *provname = OSSL_PROVIDER_name(EVP_MD_provider(md4));
        printf("EVP_MD_fetch returned unexpected success\n");

        if (OSSL_PROVIDER_available(NULL, provname))
            printf("%s provider is available\n", provname);
        else
            printf("%s provider is not available\n", provname);

        EVP_MD_free(md4);
        goto end;
    }

    ret = 0;
    printf("Success\n");
 end:
    return ret;
}

Running this test code produces this output:
EVP_MD_fetch returned unexpected success
legacy provider is not available

There was a missing cache flush on unload.


Ability to get title of GLFWwindow

I have a project where I need to get the title of a GLFWwindow, and I think this would be an easily implemented feature. I am unfortunately not very familiar with the inner workings of GLFW, so I can't implement it myself.

For now, I would just use the window user pointer as a workaround. You can store a pointer to any data you want with glfwSetWindowUserPointer and retrieve it anytime later with glfwGetWindowUserPointer.
It's probably best to put your data in a struct, so you can add more data to it later on, such as a version number or something.
Here's  an example program that stores the title and an int in the window user pointer, then when the user presses T, it retrieves the pointer and uses it to print the title to the terminal.
#include <stdio.h>
#include <GLFW/glfw3.h>

typedef struct {
    char* title;
    int myVersion;
} MyWindowData;

void key_callback(GLFWwindow* window, int key, int scancode, int action, int mods) {
    if(action == GLFW_PRESS ) {
        if(key == GLFW_KEY_T) { // READ DATA HERE
            MyWindowData* data = (MyWindowData*)glfwGetWindowUserPointer(window);
            printf("%s version %d\n", data->title, data->myVersion);
        }else if(key == GLFW_KEY_Q) {
            glfwSetWindowShouldClose(window, 1); // type Q to quit
        }
    }
}

int main() {
    if (!glfwInit()) return 1;
    char title[] = "My Cool Game";
    GLFWwindow* window = glfwCreateWindow(1280, 720, title, NULL, NULL);
    glfwSetKeyCallback(window, key_callback);
    MyWindowData data = { title, 42 };
    glfwSetWindowUserPointer(window, &data); // STORE DATA HERE
    while( !glfwWindowShouldClose(window) ) {
        // render() or whatever...
        glfwSwapBuffers(window);
        glfwPollEvents();
    }
    glfwDestroyWindow(window);
    glfwTerminate();
}

@Xydez It would not be difficult to add. May I ask why you need this?

To dynamically append things onto the title (such as adding a dot every time an action is performed for example.). It isn't really required but it's useful since the user of the API doesn't have to keep track of the title themselves.

Thanks. Will consider this for 3.4. Thought there was an earlier issue for this but can't find it now.

I must second this feature request because I know my OS (Debian with KDE) adds <2>, <3>... to end of the title to have unique window titles.
Tested on kate for screenshot.
Works same on other applications.

I have started work on this based on PR #1909 as per https://github.com/glfw/glfw/pull/1909#issuecomment-1930576696


Installation instructions

Fixing outdated installation instructions to torquebox-2.x

I looked at merging this pull request but am a bit confused as to what content has changed and what's just formatting changes. Mind resubmitting this pull request with only content changes rebased against latest 2x-dev and without the new Infinispan chapter?
We can address formatting issues in a separate pull request just so it's easier to review the 2.x doc changes - same goes for the Infinispan chapter.

Ben, to avoid mistakes I'll close this pull request and do another one with right changes.


How can I get around this?

Issue type
I'm submitting a (check one):
[ ] Bug report
[ ] Feature request
[ ] Regression (something that used to work, but stopped working in a newer version)
[x ] Support request
[ ] Documentation issue or request
ERROR in ./node_modules/angular2-json-schema-form/angular2-json-schema-form.es5.js
Module parse failed: Identifier 'Ajv' has already been declared (7:12)
You may need an appropriate loader to handle this file type.
| import { cloneDeep, filter, isEqual, map, uniqueId } from 'lodash';
| import { FormArray, FormControl, FormGroup, FormsModule, NG_VALUE_ACCESSOR, ReactiveFormsModule } from '@angular/forms';
| import * as Ajv from 'ajv';
| import { DomSanitizer } from '@angular/platform-browser';
| import { CommonModule } from '@angular/common';
ℹ ｢wdm｣: Failed to compile.

Ouh https://github.com/dschnelldavis/angular2-json-schema-form/issues/281


Using machine learning to decode animal communication

https://www.science.org/doi/10.1126/science.adg7314

editorial piece of little content, maybe interesting as a motivation in a paper


FirstOrCreate + Attrs + Where sometimes creates a new record even if one exists.

We're seeing a random (about 50% of the time) error when invoking FirstOrCreate like so:
err := tx.Where(&metadata).Attrs(NewMetadata{ID: uuid.NewRandom().String()}).FirstOrCreate(&metadata).Error

Specifically, even if the metadata in the Where is populated and exists in the database, a new record will be created in the database.
This way of using the API is straight out of the README.

Please follow https://github.com/jinzhu/gorm/blob/master/CONTRIBUTING.md

Going to close this one, please feel free to reopen it if has reproducible script.


Add ability to upload jobs in Tar format

Azkaban does not maintain execute permissions for upload shell scripts and it is cumbersome to set it in the command. Better solution is to allow for the tar format to be supported for adding jobs.

Duplicate of #42


Amazing!! Will the same weights work for "type":"segmentation" ?

I don't have a dataset with depth, so I want to train only segmentation. I'm confused, will this same architecture work?

Hi! Yes it should, just change the type argument on the config file to segmentation. Let me know if it does not work!

I think there are multiple places where that type isn't taken into consideration
I added this in dataset.py
if config['General']['type'] == "full" or config['General']['type'] == "depth":
            assert (len(self.paths_images) == len(self.paths_depths)), "Different number of instances between the input and the depth maps"

But now traindataloader also needs to handle that

it looks like there might be a problem on the structure of your folder, could you send me the structure of your working folder?

Hi, if you want to use this model only for segmentation, and if you don't have any depth data (our master branch is not very well adapted to this), you can just use the segmentation-only branch.
There are juste little modifications in the config file to bring in addition, but it should solve your issue.

Wow that works thanks,
Do you know if this model can segment the fine details with some more tweaks?


Error with distribution() if getOption("ibis.setupmessages") is NULL

Hi,
Awesome work with ibis.iSDM! I'm working on a project that involves fitting some integrated SDMs based on expert survey data and citizen science observations (from iNaturalist), and then using value of information analysis to see how much integrating the citizen science data actually informs decision making. I started playing with the package and ran into an error when using distribution().
Specifically, here's a reprex:
background <-
  system.file("extdata/europegrid_50km.tif",package = "ibis.iSDM") |>
  terra::rast()
x <- ibis.iSDM::distribution(background)
#> Error in if (getOption("ibis.setupmessages")) myLog("[Setup]", "red",  :
#>  argument is of length zero

If I use library(ibis.iSDM) before running this code, then it works fine. Alternatively, it also works if I manually specify option(ibis.setupmessages = TRUE) before running the code.  In case this is helpful, perhaps one potential fix could be changing if (getOption("ibis.setupmessages")) to if (isTRUE(getOption("ibis.setupmessages")))?
I've included my session info below, if that's useful.
> sessionInfo()
R version 4.3.1 (2023-06-16)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 22.04.3 LTS

Matrix products: default
BLAS:   /opt/R/R-4.3.1/lib/R/lib/libRblas.so
LAPACK: /usr/lib/x86_64-linux-gnu/openblas-pthread/liblapack.so.3;  LAPACK version 3.10.0

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
 [7] LC_PAPER=en_US.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

time zone: Pacific/Auckland
tzcode source: system (glibc)

attached base packages:
[1] stats     graphics  grDevices datasets  utils     methods   base

loaded via a namespace (and not attached):
 [1] terra_1.7-65       vctrs_0.6.5        cli_3.6.2          rlang_1.1.3
 [5] DBI_1.2.1          KernSmooth_2.23-21 renv_1.0.3         generics_0.1.3
 [9] assertthat_0.2.1   sf_1.0-15          zoo_1.8-12         glue_1.7.0
[13] e1071_1.7-14       fansi_1.0.6        grid_4.3.1         classInt_0.4-10
[17] tibble_3.2.1       yaml_2.3.8         foreach_1.5.2      lifecycle_1.0.4
[21] compiler_4.3.1     dplyr_1.1.4        codetools_0.2-19   Rcpp_1.0.12
[25] pkgconfig_2.0.3    ibis.iSDM_0.1.1    lattice_0.21-8     R6_2.5.1
[29] class_7.3-22       tidyselect_1.2.0   utf8_1.2.4         pillar_1.9.0
[33] magrittr_2.0.3     tools_4.3.1        proxy_0.4-27       iterators_1.0.14
[37] units_0.8-5

Just noticed a similar error with add_biodiversity_poipa() and getOption("ibis.runparallel"). Would you like me to submit a new issue? I can also keep note of any similar issues here.

Mmh...one thing that comes to my mind is setting a default during getOption("optA", default = TRUE)? I think this always returns TRUE if the option is not set/present? Maybe that could work

Nice idea! Yeah, using default = TRUE works on my comptuer too.

Now fixed with https://github.com/iiasa/ibis.iSDM/commit/b0289e1a4498380d5dfbed78884692bf88687fc8, adding default parameters for all options.
Although I would generally still recommend to attach the namespace where possible as I have not checked other functions in the package...


the nc_file must have one var and its name must be tar or pr?how can i use the rename_dic

Yes. In path_dict, it should be tas and pr.
In rename_dict, it should be like:
rename_dict = {
    `tas`: `tmp`,
    `pr`: `pre`,
}

In path_dict, it should be tas and pr, instead of the original variable names.
tas and pr have special meanings in cfr.

emm...how can I solve it

path_dict = {
    'tas': ...,
    'pr': ...,
}


On mobile screen, the vue-infinite-loading auto load all records without scrolling?

Hello, I have a problem.
.
The vue-infinite-loading works normally on Desktop screen size. (it only loads the next records when I scroll to the bottom of the screen)
.
But
.
On tablet or mobile screen size, it automatically loads all the next records until there's no record to load although I don't scroll to the bottom of the screen. That's make my web call too many API automatically and slowy.
.
Here is my example code:
infiniteHandler($state: StateChanger): void {     if (this.totalPageCnt < this.pageID) {       $state.complete();       return;     }     const payload =       this.channelId === 'bookmark'         ? { pageId: this.pageID, cnt: 1 }         : {             pageId: this.pageID,             cnt: 1,             contentsType: this.channelId ? [Number(this.channelId)] : [],           };     ArticleStateModule.loadPageAndStore(payload)       .then(() => {         $state.loaded();         this.$emit('loadingChange', false);       })       .catch((e) => {         this.$emit('loadingChange', false);         CONSTANTS.APP_ERROR.NO_DATA === e ? $state.complete() : $state.error();       });   }
Video: https://watch.screencastify.com/v/mxyYiD1i1Tfs6CzTc29O
Hope you support me soon. Thanks.
.
Sorry for my bad English

I'm experiencing the same issue
Why did you mark this issues as completed

I'm experiencing the same issue Why did you mark this issues as completed

I found my problem is because of my css, not because of this component

Thank you. I found that too
On Mon, Jun 27, 2022, 06:12 ThuongND @.***> wrote:

I'm experiencing the same issue Why did you mark this issues as completed
I found my problem is because of my css, not because of this component
—
Reply to this email directly, view it on GitHub
https://github.com/PeachScript/vue-infinite-loading/issues/321#issuecomment-1166786580,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKHL4SAWYXDWBQSCPVQUVODVREL2DANCNFSM5KUMD52A
.
You are receiving this because you commented.Message ID:
@.***>

ChananyaWinston
can you tell  me which css is affecting this?


HADOOP-17483. Remove option to enable/disable S3A magic committer . It is always on

#2636 updated the docs; this is the code-side change to remove the option of disabling magic committer support in the FS
That was a safety check to stop people enabling it on inconsistent/unguarded buckets. We can stop worrying there.
Before anyone asks: what about inconsistent third party stores: every one else's has always been consistent.
tested: s3 london with -Dparallel-tests -DtestsThreadCount=6 -Dmarkers=delete -Dtest=no -Dfs.s3a.directory.marker.audit=true -Dscale
Looks like I skipped the unit tests there...lets see what yetus says

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
36m 51s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 10 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
15m 37s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
28m 44s

trunk passed

+1 :green_heart:
compile
26m 55s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
21m  9s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
3m  8s

trunk passed

+1 :green_heart:
mvnsite
2m 29s

trunk passed

+1 :green_heart:
shadedclient
24m 41s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
1m 28s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
2m  9s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 14s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 30s

trunk passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 24s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 37s

the patch passed

+1 :green_heart:
compile
21m 12s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
21m 12s

the patch passed

+1 :green_heart:
compile
18m 29s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
18m 29s

the patch passed

-0 :warning:
checkstyle
2m 50s
/diff-checkstyle-root.txt
root: The patch generated 2 new + 51 unchanged - 1 fixed = 53 total (was 52)

+1 :green_heart:
mvnsite
2m 13s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
xml
0m  2s

The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
17m 16s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
1m 26s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
2m 10s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

-1 :x:
findbugs
1m 21s
/new-findbugs-hadoop-tools_hadoop-aws.html
hadoop-tools/hadoop-aws generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

_ Other Tests _

+1 :green_heart:
unit
10m 34s

hadoop-common in the patch passed.

+1 :green_heart:
unit
1m 23s

hadoop-aws in the patch passed.

+1 :green_heart:
asflicense
0m 48s

The patch does not generate ASF License warnings.

250m 13s

Reason
Tests

FindBugs
module:hadoop-tools/hadoop-aws

Useless control flow in org.apache.hadoop.fs.s3a.commit.CommitUtils.getS3AFileSystem(Path, Configuration, boolean)  At CommitUtils.java:Configuration, boolean)  At CommitUtils.java:[line 91]

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2637

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle

uname
Linux 0532a0f86162 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 06fef5ee43d

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/1/testReport/

Max. process+thread count
2159 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
36m 51s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 10 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
15m 37s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
28m 44s

trunk passed

+1 :green_heart:
compile
26m 55s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
21m  9s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
3m  8s

trunk passed

+1 :green_heart:
mvnsite
2m 29s

trunk passed

+1 :green_heart:
shadedclient
24m 41s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
1m 28s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
2m  9s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 14s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 30s

trunk passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 24s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 37s

the patch passed

+1 :green_heart:
compile
21m 12s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
21m 12s

the patch passed

+1 :green_heart:
compile
18m 29s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
18m 29s

the patch passed

-0 :warning:
checkstyle
2m 50s
/diff-checkstyle-root.txt
root: The patch generated 2 new + 51 unchanged - 1 fixed = 53 total (was 52)

+1 :green_heart:
mvnsite
2m 13s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
xml
0m  2s

The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
17m 16s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
1m 26s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
2m 10s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

-1 :x:
findbugs
1m 21s
/new-findbugs-hadoop-tools_hadoop-aws.html
hadoop-tools/hadoop-aws generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

_ Other Tests _

+1 :green_heart:
unit
10m 34s

hadoop-common in the patch passed.

+1 :green_heart:
unit
1m 23s

hadoop-aws in the patch passed.

+1 :green_heart:
asflicense
0m 48s

The patch does not generate ASF License warnings.

250m 13s

Reason
Tests

FindBugs
module:hadoop-tools/hadoop-aws

Useless control flow in org.apache.hadoop.fs.s3a.commit.CommitUtils.getS3AFileSystem(Path, Configuration, boolean)  At CommitUtils.java:Configuration, boolean)  At CommitUtils.java:[line 91]

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2637

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle

uname
Linux 0532a0f86162 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 06fef5ee43d

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/1/testReport/

Max. process+thread count
2159 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/1/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

I was thinking about this a bit more and worrying about "maybe I should leave the switch in but true everywhere and only document in minimal terms"
Why so? Because it is doing something unusual: renaming paths. If anyone ever tried to do a distcp from, say, hdfs and there were __magic paths in there, they'd trigger the behaviour. With a switch to turn it off for those operations,  at least if someone complains that we are doing bad things.
Note: given the hdfs rule that _ is the temp prefix I'm not over-worried, but I just think it may be good to have an emergency way to unwind

I was thinking about this a bit more and worrying about "maybe I should leave the switch in but true everywhere and only document in minimal terms"
Why so? Because it is doing something unusual: renaming paths. If anyone ever tried to do a distcp from, say, hdfs and there were __magic paths in there, they'd trigger the behaviour. With a switch to turn it off for those operations,  at least if someone complains that we are doing bad things.
Note: given the hdfs rule that _ is the temp prefix I'm not over-worried, but I just think it may be good to have an emergency way to unwind

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 20s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 10 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 12s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
25m  9s

trunk passed

+1 :green_heart:
compile
24m  6s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
19m 25s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
3m  9s

trunk passed

+1 :green_heart:
mvnsite
2m 25s

trunk passed

+1 :green_heart:
shadedclient
23m 17s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
1m 32s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
2m 10s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 10s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 28s

trunk passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 24s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 31s

the patch passed

+1 :green_heart:
compile
21m 14s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
21m 14s

the patch passed

+1 :green_heart:
compile
18m 17s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
18m 17s

the patch passed

-0 :warning:
checkstyle
2m 50s
/diff-checkstyle-root.txt
root: The patch generated 2 new + 51 unchanged - 1 fixed = 53 total (was 52)

+1 :green_heart:
mvnsite
2m 14s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s

The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
17m  6s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
1m 25s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
2m  9s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

-1 :x:
findbugs
1m 21s
/new-findbugs-hadoop-tools_hadoop-aws.html
hadoop-tools/hadoop-aws generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

_ Other Tests _

+1 :green_heart:
unit
17m  5s

hadoop-common in the patch passed.

+1 :green_heart:
unit
1m 54s

hadoop-aws in the patch passed.

+1 :green_heart:
asflicense
0m 50s

The patch does not generate ASF License warnings.

210m 32s

Reason
Tests

FindBugs
module:hadoop-tools/hadoop-aws

Useless control flow in org.apache.hadoop.fs.s3a.commit.CommitUtils.getS3AFileSystem(Path, Configuration, boolean)  At CommitUtils.java:Configuration, boolean)  At CommitUtils.java:[line 91]

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2637

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle

uname
Linux 59cf0131897a 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 06a5d3437f6

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/2/testReport/

Max. process+thread count
1287 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/2/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 20s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:

0m  0s
test4tests
The patch appears to include 10 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 12s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
25m  9s

trunk passed

+1 :green_heart:
compile
24m  6s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
compile
19m 25s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
checkstyle
3m  9s

trunk passed

+1 :green_heart:
mvnsite
2m 25s

trunk passed

+1 :green_heart:
shadedclient
23m 17s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
1m 32s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
2m 10s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+0 :ok:
spotbugs
1m 10s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 28s

trunk passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 24s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
1m 31s

the patch passed

+1 :green_heart:
compile
21m 14s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javac
21m 14s

the patch passed

+1 :green_heart:
compile
18m 17s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

+1 :green_heart:
javac
18m 17s

the patch passed

-0 :warning:
checkstyle
2m 50s
/diff-checkstyle-root.txt
root: The patch generated 2 new + 51 unchanged - 1 fixed = 53 total (was 52)

+1 :green_heart:
mvnsite
2m 14s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s

The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
17m  6s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
1m 25s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04

+1 :green_heart:
javadoc
2m  9s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

-1 :x:
findbugs
1m 21s
/new-findbugs-hadoop-tools_hadoop-aws.html
hadoop-tools/hadoop-aws generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

_ Other Tests _

+1 :green_heart:
unit
17m  5s

hadoop-common in the patch passed.

+1 :green_heart:
unit
1m 54s

hadoop-aws in the patch passed.

+1 :green_heart:
asflicense
0m 50s

The patch does not generate ASF License warnings.

210m 32s

Reason
Tests

FindBugs
module:hadoop-tools/hadoop-aws

Useless control flow in org.apache.hadoop.fs.s3a.commit.CommitUtils.getS3AFileSystem(Path, Configuration, boolean)  At CommitUtils.java:Configuration, boolean)  At CommitUtils.java:[line 91]

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2637

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle

uname
Linux 59cf0131897a 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 06a5d3437f6

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.18.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~18.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/2/testReport/

Max. process+thread count
1287 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2637/2/console

versions
git=2.17.1 maven=3.6.0 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Yes, this extra care is good to have I think. __magic might not be magic enough sometimes and people may use this path name for other purpose. I do not presume being able to disable this is definitely required; but it looks also good to me if it's possible.

Yes, this extra care is good to have I think. __magic might not be magic enough sometimes and people may use this path name for other purpose. I do not presume being able to disable this is definitely required; but it looks also good to me if it's possible.

I agree. I will switch to on, leave the ability to disable off, and change in docs "how to disable" and explain what it means/why. I'll make a separate PR as it's very different from this one

I agree. I will switch to on, leave the ability to disable off, and change in docs "how to disable" and explain what it means/why. I'll make a separate PR as it's very different from this one

Closing, superceded by #2656

Closing, superceded by #2656


VO2 MAX

Could you implement the VO2 MAX like Garmin Connect already uses for running?
I was trying to figure out a way to calculate my VO2 MAX in cycling also but the app i found doesn't support my device and i don't have a powermeter yet...

Hi,
Can you provide me the maths :) ?
Ty

I think Garmin use Firstbeat for their VO2 max estimation.
https://assets.firstbeat.com/firstbeat/uploads/2015/10/white_paper_VO2max_11-11-20142.pdf
They plot a chart of speed vs heartrate, then extrapolate to reach max heartrate and determine the maximum speed. They then have a table of activity speed to VO2 and use that to work out the theoretical VO2max.
They also split the data up into segments and ignore the ones with low speeds, erratic data etc.

@devon-badman
do you know if there is a way to calculate the cycling vo2max without a powermeter?


Connecting from GKE

Hey, I am trying to troubleshoot using alloydb-auth-proxy in GKE. I unfortunately have never been able to connect. I have an alloydb cluster/instance in the same VPC as my GKE cluster, and I am trying to use the alloydb-auth-proxy as a sidecar in my GKE cluster. However, I am getting
failed to connect to instance: Dial error: failed to dial (instance URI = "<INSTANCE>"): dial tcp <private IP>:5433: i/o timeout

Note that I also have a VM in the same VPC that I AM able to connect from using the alloydb-auth-proxy. I tried running a connectivity test, and from the GKE cluster it says "unreachable" and "Configuration analysis determines that the packet could be dropped because an internal IP address is unreachable from a public GKE cluster control plane". A connectivity test from the GKE VM succeeds, however.
I was wondering if you can share some details about how the GKE cluster needs to be configured? By chance does it require a VPC-Native cluster? Is there a way to configure a route so that it can be reached from a routes-based cluster? Or am I way off?

@cbusseata I think you'll need to deploy in a VPC-Native cluster to access your cluster.
For the standard cluster, GKE does all sorts of networking magic, so even though you're running on a VM within the same VPC, from a GKE context, there's some isolation that prevents access to the VPC.
cc @hessjcg in case he'd like to add any additional information.


[BUG]

Describe the bug
A clear and concise description of what the bug is.

When trying to reset analytics in /?analytics=on, the reset button will not apear after choosing the "I'm absolutely sure that I want to reset the results".

I think the root cause is a typo in src/streamlit_analytics2/display.py

line 153-160

            reset_prompt = st.selectbox(
                "Continue?",
                [
                    "No idea what I'm doing here",
                    "I'm absolutely sure that I want to reset the results",
                ],
            )
            if reset_prompt == "I'm sure that I want to reset the results":

The if statement in 160 has a typo that cannot match the selectbox in 157.

Expected behavior
A clear and concise description of what you expected to happen.

The button "Click here to reset" should appear

To Reproduce
Steps to reproduce the behavior:

Go to '/?analytics=on' path > Danger Zone
Click on 'I'm absolutely sure that I want to reset the results'
See error

Software Versions
Please fill in the relevant field:

streamlit_analytics2 : 0.7.5
streamlit : 1.35.0
Python3 : 3.9-slim

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

code line:
https://github.com/444B/streamlit-analytics2/blob/fe598d1814b15ff4a848e476aa053e418887fa74/src/streamlit_analytics2/display.py#L160

@mcwongmc
Thank you for raising this!
This is a really high quality bug and I can confirm that this is unexpected behavior in the current prod packages. I'll provide an RCA after a fix
I am away from my laptop but within next 24h I will release a new version 0.7.6 with a patch

@mcwongmc
Thank you again for raising this, this has been resolved in release 0.7.6, available now on pypi
The root cause (as you correctly identified) was the removal on the word "absolutely" on current line 160.
This removal was part of a flake8 / Black workflow that removed the word as it picked it up as redundant.
Thanks for identifying this!


Make Jotform integration functional

Fixes part of #32460.
Adds support for webhooks that send multipart/form-data payloads (Limitation: Files not supported yet).
Then updates the Jotform integration.
Updates to the message template and the screenshot are also included.
The 3 commits updating webhooks/common.py, test_classes.py and generate-integration-docs-screenshot can be squashd if necessary, as they have the common goal of supporting multipart/form-data in webhooks.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Looks good aside from a couple small comments.

@zulipbot add "integration review"
I've used '.multipart' as the file extension.

Looks great, merged, thanks @Niloth-p!

Updates to the message template and the screenshot are also included.

I don't see screenshots of this. Please add them to the PR description and ping me for a review.

@alya

Updates to the message template and the screenshot are also included.

I don't see screenshots of this. Please add them to the PR description and ping me for a review.

I meant the example screenshot, it's included in the file changes here, that was merged as part of the last commit.
I've now added them to the PR description as well.
I had previously proposed the italic version in #32413 (the commit wouldn't be visible now, because the branch was force pushed with a reset to exclude those commits before merging). After this review comment there on #32413, I raised a discussion in CZO regarding the same (the italics Jotform image would be available here), and followed the recommended format. I also updated there that I've updated this PR with that format just in case any corrections were desired.

Sounds good. It's helpful to have a screenshot here even if it was also posted on CZO, as well as linking to the CZO thread from the PR description if/when you start one.

The screenshot is definitely an improvement, but we should aim to give examples that make sense for businesses for any software that is commonly used for business.

It's helpful to have a screenshot here even if it was also posted on CZO, as well as linking to the CZO thread from the PR description if/when you start one.

That's true, sorry, I'll be more careful.

The screenshot is definitely an improvement, but we should aim to give examples that make sense for businesses for any software that is commonly used for business. E.g., a shortened version of one of the templates on Jotform's website would probably work fine.

Thank you for the tip, @alya!
This was generated through a shortened Jotform template too, one for a tutoring business. But, I can understand your point, in terms of the relevance to businesses that use Zulip.
I shied away from forms with any technical terms, thinking of accessibility to people unfamiliar with the field of the business and/or the English terms. And preferred a use case that uses different data types (except links and datetime values) and form elements (since we're using the data as a test fixture). Hence, I thought this was a good fit.
I'll try to think of business use cases in the future.
Would this be a preferable alternative?

A fixture with question marks could be beneficial to test the formatting.


"Escola Profissional do Infante"

@NunoD87 Please provide us with:

the school official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

@NunoD87 You might miss my previous comment. Please react.

Official website URL: http://epinfante.pt/
Offical website URL (IT): http://epinfante.pt/html/prof-curso-informatica.html

@NunoD87 Pull request merged. Thank you!


Folder structure change + Fix broken Projects

Description
This PR intends to create a folder structure which is easy to navigate and clean. While moving the folders around, it intends to separate the Source and the Tests of FreeRTOS-Plus-TCP.
Both changes mentioned above will break the TCP test project. Therefore, the fix for the broken project is also included in the same PR.
The folder structure which the PR intends to achieve:
FreeRTOS
FreeRTOS-Plus
        + Demo
        + Source
                + FreeRTOS-Plus-<Lib_name>
                + FreeRTOS-Plus-TCP -> Remove the ‘test’ folder and take it to <root>/Test/FreeRTOS-Plus-TCP/Integration/Full-TCP-Networkless
        + Test
                + CBMC
                + Cmock
                + FreeRTOS-Plus-TCP
                        + Integration
                                + Full-TCP-Networkless -> add the above mentioned ‘test’ folder here
                        + Unit

There are 3 commits in the PR:

First one cleans up the folders and creates new ones
Second one fixes the Projects which broke due to folder structure change
Third one moves the test folder to the ideal location

Test Steps

Clone the repository git clone https://github.com/AniruddhaKanhere/FreeRTOS.git --recurse-submodules ./Folder
Checkout the branch: git checkout Folder_structure_change
Go to FreeRTOS-Plus\Test\FreeRTOS-Plus-TCP\Integration\Full-TCP-Networkless and open Full-TCP-Networkless.sln.
Edit the configNETWORK_INTERFACE_TO_USE in FreeRTOS_Config.h.
Compile and run! 4 tests should run...

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Here's what I have with tree for ./Test/FreeRTOS-Plus-TCP/Integration
├── Integration
│   └── Full-TCP-Networkless
│       ├── Full-TCP-Networkless.sln
│       ├── ReadMe.txt
│       ├── Read_Me_Build_Instructions.url
│       ├── Source
│       │   ├── Config\ Files
│       │   │   ├── FreeRTOSConfig.h
│       │   │   └── FreeRTOSIPConfig.h
│       │   ├── Test_runner
│       │   │   ├── test_runner.c
│       │   │   ├── test_runner.h
│       │   │   └── test_runner_config.h
│       │   ├── demo_logging.c
│       │   ├── demo_logging.h
│       │   ├── main.c
│       │   └── printf-stdarg.c
│       ├── WIN32.vcxproj
│       ├── WIN32.vcxproj.filters
│       ├── WIN32.vcxproj.user
│       ├── WinPCap
│       │   ├── Packet32.h
│       │   ├── PacketData.h
│       │   ├── Win32-Extensions.h
│       │   ├── arch.c
│       │   ├── bittypes.h
│       │   ├── ip6_misc.h
│       │   ├── netif.h
│       │   ├── pcap
│       │   │   ├── bluetooth.h
│       │   │   ├── bpf.h
│       │   │   ├── namedb.h
│       │   │   ├── pcap.h
│       │   │   ├── sll.h
│       │   │   ├── usb.h
│       │   │   └── vlan.h
│       │   ├── pcap-bpf.h
│       │   ├── pcap-namedb.h
│       │   ├── pcap-stdinc.h
│       │   ├── pcap.h
│       │   ├── remote-ext.h
│       │   └── wpcap.lib
│       ├── dlldata.c
│       └── test
│           ├── freertos_tcp_test_access_declare.h
│           ├── freertos_tcp_test_access_dns_define.h
│           ├── freertos_tcp_test_access_tcp_define.h
│           └── test_freertos_tcp.c

I'd like to propose these changes:

No space in folder name. e.g. "Config Files"
In file name and folder name, try to stick with either dash '-' or underscore '_' for things we have control for. (There's not much we can do about for WinPCap dir.) But the rest, may want to pick a style.
Can we get rid of another layer of Source and test? This is confusing to people. The entire directory is already in ./FreeRTOS-Plus/Test/FreeRTOS-Plus-TCP. To a reader, how to interpret source under test and test under test? A possible alternative could be "test" --> "test_cases" and flatten "Source". (Or whatever makes most sense.)

Sorry for being picky.

@yuhui-zheng, I have made some changes taking your comments in consideration. Can you have a look at the revised folder structure?


image

![](bg_fb_cover.jpg)

dont show anything on my end. the image is already in the same folder (home). I am using default template, markdown and twig is on.

No clue what you are asking, i just see a broken image in single ticks...

No idea what your asking here. I only see a broken image in single ticks.

exclamation braces then  filename in parenthesis
im putting the code but markdown processes it as image which cannot be found here

Please use back ticks around your code ( ```)

im trying the sample of the markdown as well but not processing on grav https://guides.github.com/features/mastering-markdown/

![](bg_fb_cover.jpg)

thanks for the time looking at this... been a headache... i tried putting it on images folder and calling the path but same thing happens

if you have an image called my-image.jpg in your page's folder.. ie:
01.home
  - default.md
  - my-image.jpg

then in default.md you can just have:
# title

![](my-image.jpg)

This should just work

# All Workers Unity
![](bg_fb_cover.jpg)
# ****
All Workers Unity (AWU) is a national campaign network uniting all sectors for the implementation of a National Minimum Wage in the amount of P750 daily for the private sector & P16,000 monthly for the public sector and --the Banning of Contractualization in PH.
****

you can check out this at allworkersunity.com
files in the 'home' folder are default.md and bg_fb_cover.jpg

try this:
# All Workers Unity

![](bg_fb_cover.jpg)

# ****
All Workers Unity (AWU) is a national campaign network uniting all sectors for the implementation of a National Minimum Wage in the amount of P750 daily for the private sector & P16,000 monthly for the public sector and --the Banning of Contractualization in PH.
****

BTW, you can't put HTML inside a Markdown element, so your # **** <p style... will not be displayed as you expect, just use
 tag if you plan to use other HTML mixed in there.  Also you can't put a
 tag in an
 tag anyway, that's invalid HTML.

its working on the website http://allworkersunity.com
seems strange that image dont work

ok, guess that mixing Markdown does work there.. weird as it's not technically valid.
Anyway, regarding the image i see a possible problem.  It's generating retina images for this image, but it's not using the cached location:

I'm not sure how this happens, as it doesn't do this in my testing.  Are you sure you just using ![](bg_fb_cover.jpg) and not using derivatives: https://learn.getgrav.org/content/media#sizes-with-media-queries ??

BTW, it does appear that you have some 'retina' image script running outside the Grav retina capabilties.  This is adding support for various retina image sizes and images that don't exist, for example, my browser is trying to load http://allworkersunity.com/user/pages/home/bg_fb_cover-960.jpg which does not exist (hence the broken image), but the actual src image is there:
http://allworkersunity.com/user/pages/home/bg_fb_cover.jpg

yes no other derivatives just plane insert image.  i tried changing image using http its not showing. btw my image cache is off and the overall cache is off as well -devt stage.

i tried turning on the image cache and cleared the image cache... same thing

if its retina problem how do i turn it off since i dont have any derivatives?

I have no idea how you are getting that derivative based output without manually setting ?derivatives on the markdown image URL.
I can't replicate this locally. I just add an image locally, and i just get a regular IMG tag output. Maybe you can zip up your site and let me download and test with it to see what's going on?

i cant upload the .md file nor the zip files of user folder

upload it to dropbox/googledrive/ whatever.  I can get the image, i need your site, as you seem to have something going on beyond the page and regular grav stuff.  I can't replicate your issue locally, so need as much of your site as possible to try to recreate it.

https://drive.google.com/file/d/0B5nNDrg-kh_2R0cxeXJBQ2tVTGM/view?usp=sharing

Thank you in advance

The problem is caused by the imgsrcset plugin, removing it fixes the problem https://github.com/olevik/grav-plugin-imgsrcset/issues - tagging @OleVik as he knows better what could be the problem

Thanks for notifying me @flaviocopes. The code @rhukster highlighted;

points to the ImgSrcset-plugin being active, indeed. It should be noted that this does not actually generate the various image-sizes for you, only the srcset-attribute. Thus you would need to generate the images yourself (see this gulpfile for an example).

Thanks for tracking that down guys. Makes sense.  That imgsrcset plugin creates the appropriate HTML output to support various imagest but as ole says, it's your responsibility to create those images.


Not working with react/http-client >= 0.4.13

Q
A

Bug?
yes

New Feature?
no

Version

Actual Behavior
This library return a error when creating an response
Expected Behavior
Working :)
Steps to Reproduce
Use last version of react/http-client
Possible Solutions
We should rework the way the response is build, it's no longer working since this PR has been merged into react/http-client https://github.com/reactphp/http-client/pull/66

I'll take a look today 😄.


Robotiq FT300 sensor drivers, urdf, models, meshes

Do these exist anywhere? Can I use the FT150 driver for the FT300?

@nicolaslauzier (robotiq) is probably the best one to ask.

The FT300 communicates exactly like the FT150 and as such, the same driver can be used for both sensors. I cannot comment for the urdf, models and meshes as these are from the community (Robotiq provides CAD model, but nothing in a ROS-compatible format).
Hope this helps.

I've made a PR adding the FT300.

#98


First pass at correcting new lines in the output and some logs not ma…

…king it to the output and debug consoles.

Pushed updated related to our slack conversation. Should be good for review again.


help needed: value of node on mouse over

Hi, how can I get the value of the node (or the ID of it) inside a dyn action reacting to the mouseover event?
Many thanks in advance!

solved.


Optiqwill and Reflette additions

also Ledian tier fix

there are conflicts to resolve on commit, first conflict is moving Ledian to ZU, second is adding the mons tiering at the bottom. for both, you can select the changes from this branch

It's an easy conflict to fix, so most of the pull request is atleast still valid

It's an easy conflict to fix, so most of the pull request is atleast still valid

In fact, I just fixed the conflicts myself.


BasePathDetector::getBasePathFromApache returns wrong path

Hi,
I had an issue finding the correct base path for my Slim 4 App with this tool.
The method getBasePathFromApache might fall one directory short of reality.
In a case, where the index.php to setup the app is called two or more directories below the webroot, a base path is returned, where the last directory is missing.
Example:
Calling the app like this: http://localhost/projects/exampleapp/api/
The intended result is "/projects/exampleapp/api", which might end up as "https://example.com/api" in production.
But it returns: "/projects/exampleapp".
This is caused by the duplicate use of "dirname()" on the scriptname in line 86 of the file BasePathDetector.php.
After reducing the line to a single call of "dirname()" the result is correct. At least in my use case.
What is the intention of calling dirname twice?
Which sideeffects would it cause to ommit one call?
Best regards,
Hagen

The first namename removes the script filename: index.php and the second call removes the public/ directory from to get the basepath.
<?php

$serverScriptName = '/slim4-skeleton/public/index.php';

$scriptName = str_replace('\\', '/', dirname(dirname($serverScriptName)));

echo $scriptName; // Output: /slim4-skeleton

https://3v4l.org/KHWRD
The issue in this specific case is, that this basepath detector doesn't support more than one sub-directory.

Any feeda


Update documentation to include min req

.. for using manual profiling.

@juliusl : Thanks for your contribution! The author, @cweining, has been notified to review your proposed change.

#sign-off

@rjagiewich : Thanks for your contribution! The author, @cweining, has been notified to review your proposed change.


Fix Typos in XML docs

PR Details
I was looking through the collections and threading documentation and noticed some typos, this should fix the ones I found.
Motivation and Context
Documentation clarity is useful.
Types of changes

[x] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] I have built and run the editor to try this change out.

Thanks !


Question about Trackr

Previously, I would like to congratulate on your Go-Hackathon victory :tada:
After I learn your README and pitching deck, I'm so curious more about Trackr or sort of it that the price is as low as $1 when purchased in mass scale. Would you mind to share the link or info about cheap tracker device (I assume the price is $1 when purchased in mass scale) that support with Go-Track, has Crowd Locate feature (like TrackR does), and has battery last for 1 year? AFAIK, currently TrackR cheapest price is $12 (purchased in mass scale).
Regards :beers:

@pewh Thanks for that, we really appreciate it. Thank you as well for having interest in our project.
One example would be https://www.alibaba.com/product-detail/Mobile-Phone-Application-bluetooth-beacon-and_1970164099.html?spm=a2700.7724838.0.0.DYpLXu&s=p. They claim to have 50-90 meters range and 1-2 years battery life, but we didn't actually contact them for we do not plan to mass-produce it in the near future.
TrackR sells it for $12. We would never know how much does it take to build each of them. Moreover, it has additional feature such as Item Ringer (at least it must have mini speaker inside). The $1 price is only for the plain beacon + battery, while for production we have to think about packaging etc. But knowing TrackR sells it for $12, we now know that the production cost is (way) lower than that.


Support optional periodic tests

Allow authors to configure optional period tests for their solution repos. For lint we can likely inject a cron in GHA. For int we could orchestrate trigger exec via scheduler. We might need to brainstorm how we want failures to be surfaced back, maybe GH issues?

For lint we can likely inject a cron in GHA.

@bharathkkb - Do we want to do automatically for every repo, or expose as a bool in https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/blob/master/infra/terraform/test-org/org/locals.tf#L57?
Something like actions/github-script can be used to open a issue for the linter.

@apeabody I think we should add it as an optional bool but disabled by default.

Something like actions/github-script can be used to open a issue for the linter.

I was thinking of a unified way for both lint and int failures. Maybe we can have another cron that runs from main ~1h after nightly cron and gets the github check status using  actions/github-script.


Overhaul CLI argument forwarding to processes started by the editor

First, this reverts a the changes about making the editor forward certain CLI arguments to new processes (either a new instance of the editor itself, the project manager or the running project), such as those in b2d27214aa0bee3cf0dc091846bf087e007c9607, 2b292a1a2ad7c0a74ea186d9f53afa70b65d390f and 116f03a1b68ecc59f9755cd0219e256c63955f55.
Then, it adds a little framework that makes the management of argument forwarding more centralized, complete and maintainable.
P. S.: I said I would do this as a Saturday morning project, but it turned out to be a Saturday evening one.
UPDATE: Version for 3.5+ in #64375.

Thanks!


Python 3 support

Trying to port the code to support both Python 2 and 3. The tool 2to3 was used. Most of the manual fixes are related to Unicode strings in Python 2.
Basically, pexpect.spawn() by default works in the byte-string interface in Python 3. As recommended by the pexpect manual, the option encoding='utf-8' is used to decode data. Consequently, the expected strings in the unit test are manually specified as Unicode.
Was having issues with Travis CI for mininet/test. Retesting...

Still getting one error related to pexpect:
testBasic (test_walkthrough.testWalkthrough)
Test basic CLI commands (help, nodes, net, dump) ... Exception AttributeError: "'spawn' object has no attribute 'closed'" in <bound method spawn.del of <pexpect.spawn object at 0x7f8cb57b6b50>> ignored
ERROR
Anyone has an idea?

This is interesting; offhand I don't know why pexpect would fail, but I will take a look at it; I think I've sometimes seen things like that when an object is of the wrong type or is None.
Some things I'm wondering about:

Is utf8 a python3 related issue or is it orthogonal? I believe strings are utf8 by default in python3, but will expect just work properly out of the box or do we need changes for that?

Do items() and keys() really need to be converted to lists? That seems odd, but maybe it's needed.

You are right, strings are utf8 by default. pexpect returns byte string unless when encoding is specified. So the current unittest fails in Py 3 because byte-strings cannot compare with unicode string.
After passing encoding='utf8' to pexpect, it will work in Py 3 but fails in Py 2. I need to explicitly enforce the expected strings to be utf8-encoded with u'. It follows the practise suggested here: http://python-future.org/compatible_idioms.html#strings-and-bytes
The odd conversions of items() and keys() are done by 2to3 automatically. I can fix them manually.

@cuihantao Thanks for your work on this!!
I have started working on a version of python3 compatibility in https://github.com/lantz/mininet/tree/py3-experimental

Instead of using six my approach is to add a bit of compatibility glue in util.py.
The pexpect changes are really annoying - which is why I am thinking of a patched pexpect.spawn in util.py
For MiniEdit.py I like your changes and I think we can just use them as is
pmonitor is hard to get right ;-p
I wonder what we should do as far as installing Mininet libraries for both python 2 and 3

Yeah, I'm not quite sure what changes (if any) should be made to install.sh and/or make install.
I am thinking that since mininet runs with whatever the default python is, perhaps installation should do the same.
An alternative idea would be to try to install for python2 and python3 if they are both available. One thing that's a bit challenging about install.sh is that it installs using sudo, which tends to reset $PATH and won't use the user's bash aliases if they are set one way or another.
For the Mininet VM I think we should possibly/probably have both python2 and python3 versions.


DOC: Reference the bibliography file correctly.

Reference the Bibliography
file correctly: Bibliography.bib instead of Bibliography.tex.
Link it to the actual file.

The name change is Ok. Not so sure about adding a link though, it may cause confusion about which one you are working on. E.g if you have checked out the file that is the one you should work on.

At this stage, just change the extension. I can do that if you want.
Once all the chapters are entered then there is no need for this section.

@ajpmaclean Kept the filename only in 8a3119d.

Thanks John


Buildasaur

Buildasaur - Automatic testing of your Pull Requests on GitHub and BitBucket using Xcode Server. Keep your team productive and safe. Get up and running in minutes. @buildasaur :large_orange_diamond:

@czechboy0 on what category o you think we can add?

Tools or Testing?