${subsystem}_responses_sent metic resets during "planned periodic restart of DNS server"

Hello,
Thanks again for the amazing DNS project.  Just wanted to comment on a low priority issue (maybe it's by design?) that seems to have started with commit 6a383ba of v2.0.0-beta.
During the daily "planned periodic restart of DNS server" the ${subsystem}_responses_sent metric now resets to 0.  Previously this was persisted during these restarts and only a restart of the full container would reset this number.
Logs are below from my setup.  Every 24 hours during that planned restart this metric is resetting.
Thanks again!
========================================
========================================
================= dns ==================
========================================
=========== Made with ❤ by ============
======= https://github.com/qdm12 =======
========================================
========================================

Running version v2.0.0-beta built on 2022-06-27T14:07:18Z (commit 6a383ba)

🔧 Need help? https://github.com/qdm12/dns/discussions/new
🐛 Bug? https://github.com/qdm12/dns/issues/new
✨ New feature? https://github.com/qdm12/dns/issues/new
☕ Discussion? https://github.com/qdm12/dns/discussions/new
💻 Email<EMAIL_ADDRESS>💰 Help me? https://www.paypal.me/qmcgaw https://github.com/sponsors/qdm12
2022-09-25T17:32:13Z INFO Settings:
├── DNS upstream connection: dot
├── DNS server listening address: :53
├── DNS over TLS:
|   ├── DNS over TLS providers: cloudflare and google
|   ├── Plaintext fallback DNS providers: cloudflare
|   ├── Request timeout: 3s
|   └── Connecting over: IPv4
├── Cache:
|   ├── Type: lru
|   └── LRU cache:
|       └── Max entries: 10000
├── Filtering:
|   ├── Blocked categories:
|   └── Rebinding protection: enabled
├── Logging:
|   └── Level: INFO
├── Middleware logging: disabled
├── Metrics:
|   ├── Type: prometheus
|   └── Prometheus:
|       ├── Listening address: :9090
|       └── Subsystem: dns3
├── Check DNS: enabled
└── Periodic update: every 24h0m0s
2022-09-25T17:32:13Z INFO using DNS address <IP_ADDRESS> internally
2022-09-25T17:32:13Z INFO [prometheus server] listening on :9090
2022-09-25T17:32:13Z INFO [healthcheck server] listening on <IP_ADDRESS>:9999
2022-09-25T17:32:13Z INFO starting DNS server
2022-09-25T17:32:13Z INFO DNS server listening on :53
2022-09-25T17:32:13Z INFO restarting DNS server the first time to get updated files
2022-09-25T17:32:13Z INFO downloading and building DNS block lists
2022-09-25T17:32:13Z INFO 0 hostnames blocked overall
2022-09-25T17:32:13Z INFO 0 IP addresses blocked overall
2022-09-25T17:32:13Z INFO 13 IP networks blocked overall
2022-09-25T17:32:13Z INFO starting DNS server
2022-09-25T17:32:13Z INFO DNS server listening on :53
2022-09-26T13:13:21Z WARN dial tcp <IP_ADDRESS>:853: i/o timeout
2022-09-26T17:01:16Z WARN dial tcp <IP_ADDRESS>:853: i/o timeout
2022-09-26T17:01:16Z WARN dial tcp <IP_ADDRESS>:853: i/o timeout
2022-09-26T17:32:13Z INFO planned periodic restart of DNS server
2022-09-26T17:32:13Z INFO downloading and building DNS block lists
2022-09-26T17:32:13Z INFO 0 hostnames blocked overall
2022-09-26T17:32:13Z INFO 0 IP addresses blocked overall
2022-09-26T17:32:13Z INFO 13 IP networks blocked overall
2022-09-26T17:32:13Z INFO starting DNS server
2022-09-26T17:32:13Z INFO DNS server listening on :53
2022-09-27T17:32:13Z INFO planned periodic restart of DNS server
2022-09-27T17:32:13Z INFO downloading and building DNS block lists
2022-09-27T17:32:13Z INFO 0 hostnames blocked overall
2022-09-27T17:32:13Z INFO 0 IP addresses blocked overall
2022-09-27T17:32:13Z INFO 13 IP networks blocked overall
2022-09-27T17:32:13Z INFO starting DNS server
2022-09-27T17:32:13Z INFO DNS server listening on :53
2022-09-28T17:32:13Z INFO planned periodic restart of DNS server
2022-09-28T17:32:13Z INFO downloading and building DNS block lists
2022-09-28T17:32:13Z INFO 0 hostnames blocked overall
2022-09-28T17:32:13Z INFO 0 IP addresses blocked overall
2022-09-28T17:32:13Z INFO 13 IP networks blocked overall
2022-09-28T17:32:13Z INFO starting DNS server
2022-09-28T17:32:13Z INFO DNS server listening on :53
2022-09-29T17:32:13Z INFO planned periodic restart of DNS server
2022-09-29T17:32:13Z INFO downloading and building DNS block lists
2022-09-29T17:32:13Z INFO 0 hostnames blocked overall
2022-09-29T17:32:13Z INFO 0 IP addresses blocked overall
2022-09-29T17:32:13Z INFO 13 IP networks blocked overall
2022-09-29T17:32:13Z INFO starting DNS server
2022-09-29T17:32:13Z INFO DNS server listening on :53

Here's a grafana screenshot that goes along with these logs.

I didn't test it yet, but 903b287471a9966d2111f0bb3e0d657024c1d1e0 should fix it, it now caches metrics across restarts (a bit of an ugly Go piece of code, but I couldn't think of something else or downside in doing so). Feel free to try the v2.0.0-beta image again, otherwise I'll run it someday soon and close it if resolved. Thanks 👍 !!


⚠ Bofrost.fr has degraded performance

In 2ce66bc, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 897 ms

Resolved: Bofrost.fr performance has improved in 3035cdf after 6 minutes.


Can not unbind docker container from port 80

Let me begin by saying I am not too familiar with the docker process. I followed your steps to get the container up and running. I should mention I have development applications being served from apache on port 80 as well and I can hit several of those. After successfully hitting the app in a browser, (unfortunately from one of my predefined localhost ServerNames) that particular site always hits the bound docker port and I get a screen of "Be right back.".  I have tried several things to resolve this: removing all docker images, killing docker processes, restarting my machine, etc. What simple step am I missing here?

Figured this out and the port binding was not the case. Closing.

Awesome :)


[CSDiagnostics] Look through l-value conversions when mismatch is in …

…subscript setter
repairFailures needs a special case when l-value conversion is
associated with a result type of subscript setter because otherwise
it falls through and treats result type as if it's an argument type,
which leads to crashes.
Resolves: rdar://84580119

@swift-ci please smoke test

@swift-ci please smoke test macOS platform


SDK seems to create corrupted workbooks.

Description
Creating the "sample minimal" Excel spreadsheet with the SDK seems to create a corrupted file (Excel doesn't like it). If I add some row data I still get the issue.
Information

.NET Target: .NET Core (although I could also repro it with .NET Framework)
DocumentFormat.OpenXml Version: 2.10.0

Repro
using (SpreadsheetDocument document = SpreadsheetDocument.Create("D:\\Diff\\Sample2\\Foo.xlsx", SpreadsheetDocumentType.Workbook, false))
            {
                // A SpreadsheetDocument must have at least a WorkbookPart and a WorksheetPart
                WorkbookPart workbookPart = document.AddWorkbookPart();
                workbookPart.Workbook = new DocumentFormat.OpenXml.Spreadsheet.Workbook(); // Name ambiguity...

                WorksheetPart worksheetPart = workbookPart.AddNewPart<WorksheetPart>();
                worksheetPart.Worksheet = new Worksheet(new SheetData());

                Sheets sheets = workbookPart.Workbook.AppendChild(new Sheets());

                Sheet sheet = new Sheet() { Id = workbookPart.GetIdOfPart(worksheetPart), SheetId = 1, Name = "Test" };
                sheets.Append(sheet);

                SheetData sheetData = worksheetPart.Worksheet.AppendChild(new SheetData());

                ////Row row = new Row();

                ////row.Append(
                ////    ConstructCell("Id", CellValues.String),
                ////    ConstructCell("Name", CellValues.String),
                ////    ConstructCell("Birth Date", CellValues.String),
                ////    ConstructCell("Salary", CellValues.String));

                ////// Insert the header row to the Sheet Data
                ////sheetData.AppendChild(row);

                ////uint id = 1;
                ////foreach (Spreadsheet spreadsheet in m_sheets.Values)
                ////{
                ////    Debug.Assert(spreadsheet != null);

                ////    spreadsheet.SaveToExcel(workbookPart, sheets, id++);
                ////}

                workbookPart.Workbook.Save();

                document.Close();
            }

Observed
Excel complains when trying to open the file, message:
We found a problem with some content in 'Foo.xlsx'. Do you want us to try to recover as much as we can? If you trust the source of this workbook, click Yes.
Expected
The workbook opens just fine in Excel.

It works for me if you remove the line (SheetData is already added to Worksheet when it was passed in as an argument to the constructor so this extra line causes it to be added twice):

SheetData sheetData = worksheetPart.Worksheet.AppendChild(new SheetData());

and also replace workbookPart.Workbook.Save() with document.Save(). Where did you get this sample from? If it is in the github repo it should be updated otherwise this should have been a question on stackoverflow.

That did it, thank you.
I'll contact the owner of the page where I copied this from (after making sure it wasn't my copy/paste) mistake.
Thanks!


[Bug]: Players do not spawn on their bed when entering a world

/mv version -p output
/mv tp world
Server logs
n/a
Server Version
This server is running Paper version git-Paper-409 (MC: 1.19.3) (Implementing API version 1.19.3-R0.1-SNAPSHOT) (Git: 8427c88)
You are running the latest version

Bug Description
Set bed down, went to another world, tried to go back to world where bed was placed, spawned at world spawn instead of bed. I checked all configs to make sure my settings were correct, I found that they were. I removed plugins that might conflict with this plugin, still having issues. I was able to spawn at bed typing /mvtp b:playerbed.
Steps to reproduce
Set bed down, went to another world, tried to go back to world where bed was placed, spawned at world spawn instead of bed.
Agreements

[X] I have searched for and ensured there isn't already an open or resolved issue(s) regarding this.
[X] I was able to reproduce my issue on a freshly setup and up-to-date server with the latest version of Multiverse plugins with no other plugins and with no kinds of other server or client mods.

How are you going between the worlds? Using /mv tp command or portals or something else?

/mv tp

i have this exact issue. ran my own tests and same conclusion.


"window is undefined, vue-test-utils needs to be run in a browser environment" error in Vue project

I'm trying to run Stryker on a Vue project with webpack and after fixing some other issues -mainly npm package errors- I'm stuck with this error:
Error: [vue-test-utils]: window is undefined, vue-test-utils needs to be run in a browser environment.  You can run the tests in node using jsdom
I've created a quick-and-dirty example at https://github.com/glezalex/vue-tdd to show an example of this error, in case somebody could help me understand where this issue comes from :).

@glezalex thanks for opening this issue.
@simondel or @Archcry could one of you take a look? You guys are vue experts, right?

Glad to see some Vue people around here! Thanks for the repro repo :) I'll check it out tomorrow ♥

@simondel thanks a lot! I'm playing around with your fork and it works like a charm. Also your commits are of great help to me to learn some tricks.

No problem! I've also submitted my changes as a PR to your repo :)
I've reopened this issue because the error is still there if I use the mocha test runner programmatically from Stryker.

Alright, so the problem is this:
You load jsdom-global in test/setup.js. This file is loaded by your test/mocha.opts file:
--file test/setup.js

The file is in the default location so it's always loaded, however, Stryker doesn't do anything with your specific mocha.opts file because:

The only supported mocha options are used: --ui, --require, --async-only, --timeout, --grep (or their short form counterparts). Others are ignored by the @stryker-mutator/mocha-runner.

(as per the @stryker-mutator/mocha-runner readme)
Stryker should log that it ignores the option because of this line of code:
https://github.com/stryker-mutator/stryker/blob/master/packages/mocha-runner/src/MochaOptionsLoader.ts#L85
but for some reason it doesn't do that..
In my local file system I've changed the logging to warning and then it does show the logging:

To sum it up
We have a few issues:

We don't load files that are specified in the mocha.opts file. We could just load these to your mocha files before we load the regular mocha files and we should be fine.
The logging that warns you of ignored properties doesn't show at all
The logging is debug logging. Since it results in us running in a different setup then a regular testrun I would expect it to be logLevel warn.
The logging shows the key of the option, not the entire option. I think the message Ignoring option "--file test/setup.js" as it is not supported. would be more clear than the one you can see above in the screenshot.

@nicojs What are your thoughts on my remarks?
@glezalex Thanks for making us aware of this! Feel free to take another look at my fork, I've updated it with the latest config 👍

Also, I noticed #1433 because of this issue 👍

What's the difference between --file and --require. I honestly dislike the mocha.opts file a lot. Why don't the mocha guys provide an API to load options. Its very hard to keep up with is.
Why not?
I disagree. For example --exit should just be ignored, as it doesn't make sense for Stryker. Logging these things on warn forces people to maintain 2 sets of configuration
Agreed

Thanks again @simondel, I've already incorporated your fixes to my repo and it really exceeds what I expected -and needed 😄-. I'm working with another colleague on a very basic example for an event in which we would like to show stryker in action and this is of great help.

--file <file/directory/glob>
Explicitly include a test file to be loaded before other test files files. Multiple uses of --file are allowed, and will be loaded in order given.
Useful if you want to declare, for example, hooks to be run before every test across all other test files.
Files specified this way are not affected by --sort or --recursive.
Files specified in this way should contain one or more suites, tests or hooks. If this is not the case, consider --require instead.
--require <module>, -r <module>
Require a module before loading the user interface or test files. This is useful for:
Test harnesses
Assertion libraries that augment built-ins or global scope (such as should.js)
Instant ECMAScript modules via esm
Compilers such as Babel via @babel/register or TypeScript via ts-node (using --require ts-node/register)
Modules required in this manner are expected to do work synchronously; Mocha won't wait for async tasks in a required module to finish.
Note you cannot use --require to set a global beforeEach() hook, for example — use --file instead, which allows you to specify an explicit order in which test files are loaded.

I'm able to run Stryker on the vue-tdd repo as well if I change
--file test/setup.js into --require ./test/setup.js (the relative path is important, since it's a local file)

I'm working with another colleague on a very basic example for an event in which we would like to show stryker in action and this is of great help.

Just be sure to send pics or even tweet about it, we'll retweet (@stryker-mutator)
@simondel
Thanks for the documentation. I guess we should parse & support --file as well. It is too bad that we need to keep this functionality in sync. We could also log all unknown options on warn level. So we would keep logging --exit on debug, but --some-fancy-feature would be logged on warn
@glezalex would you mind if we change the name of this issue to reflect this new functionality? "Support --files in mocha.opts" (or something).

We'll for sure, thanks @nicojs.
Yes sure, if finally better reflects the actual issue so the original name I gave it could be misleading.


[Go Live][Nov 25]

Push to live

Learn Build status updates of commit c8db5e8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

msteams-platform/apps-in-teams-meetings/build-extensible-conversation-for-meeting-chat.md
:white_check_mark:Succeeded
View

msteams-platform/assets/images/conversation-bot/clone-repository.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/conversation-bot/manifest-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/conversation-bot/project-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/conversation-bot/upload-manifest-zip.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/appsettings-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/clone-repository.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/dotnet-run-command.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/manifest.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/manifest-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/project-file.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/project-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/sbs-messagingextension-action/upload-manifest-zip.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tabs/location-picker-desktop.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/tabs/location-picker-mobile.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-file-upload-bot/appsettings-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-file-upload-bot/clone-repository.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-file-upload-bot/dotnet-run-command.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-file-upload-bot/manifest-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-file-upload-bot/manifest-zip-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-file-upload-bot/project-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/teams-file-upload-bot/upload-manifest-zip.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/appsettings-file-location.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/Teams-link-unfurling/clone-repository.png
:white_check_mark:Succeeded
View

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


gate: allow disabling max concurrent

Allow disabling max concurrency limit by setting limit to <= 0. Previously it was impossible to disable this limit.
Signed-off-by: Giedrius Statkevičius<EMAIL_ADDRESS>

I doubt it makes much sense to set the limit to 0 as it is because it effectively disables everything :smile: what would be the use-case for that? :shrug:

In Golang the zero value is usually used to indicate no limit, e.g. setting a timeout of 0 would mean no timeout etc.. Also as you pointed out, making the semantics of 0 equal to "don't allow any request" makes no sense. This is why I thought we can use 0 to disable the limit my instantiating a noop gate.


[REVIEW] UtkarshMishra12

GitHub Profile Link:
@UtkarshMishra12
You can also join the Discord community here
Feel free to check out other cool repositories of the EddieHub Community here

Only one thing from my side consider removing the view count due it's inaccurate

Thank you for sharing, we will review on a live stream soon 👍

Maybe center align the logs in the Tech Stack section and add all of them in one line if possible, instead of 3?

Maybe center align the logs in the Tech Stack section and add all of them in one line if possible, instead of 3?

Actually, it's fine if it's 3 lines, as the first line is coding languages, second is game engines, and third if IDE's. But, I think it will look better with a center align :)

Looks great! Some thoughts

Make use of the GitHub profile description (left)
I think visitor badge is not needed
Keep badges the same size and style
Maybe change the repository list link to your favourite / current repo


Update SoCsAndCustomBoards.md

Docs Build status updates of commit 16b7506:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/learn-about-hardware/SoCsAndCustomBoards.md
: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.


docs: fix typo in url to vault k8s auth

Description
The link was broken because it linked to the GitHub repo:
https://github.com/argoproj-labs/argocd-vault-plugin/blob/v1.12.0/docs/vaultproject.io/docs/auth/kubernetes#configuration
Checklist
Please make sure that your PR fulfills the following requirements:

[x] Reviewed the guidelines for contributing to this repository
[x] The commit message follows the Conventional Commits Guidelines.

Type of Change

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[x] Documentation content changes
[ ] Other (please describe)

Codecov Report

Merging #398 (4ee5d06) into main (4646f51) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #398   +/-   ##
=======================================
  Coverage   77.60%   77.60%
=======================================
  Files          22       22
  Lines        1009     1009
=======================================
  Hits          783      783
  Misses        142      142
  Partials       84       84

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Sigma Prime Audit

What has been done

Audit Report
Brownie Testing Artifacts

Additional Comments
Going to request a review from @zedt3ster before merging to check the audit report.

An updated version of the audit was performed


Document the meaning of output

I tried this but I don't understand what the output means. It'd be helpful if it was documented.

The output is documented here. Not sure if writing new documentation is useful but we should at least link to it somewhere.

Ah, thank you! Yes, link would be good.


GUI - Undefined variable $slot

First take a look at the following

https://github.com/jeremykenedy/laravel-roles#opening-an-issue
https://github.com/jeremykenedy/laravel-roles/issues?q=is%3Aissue+is%3Aclosed

Describe the bug
When trying to open the route /roles and the route /permissions, it sends the error "Undefined variable $slot".
Screenshots

Desktop

OS: Windows 10
Browser: chrome
Laravel: 9.43
PHP: 8.1.10

Additional context
New laravel app + breeze.

That screenshot doesn't look like it's this package.

You need to replace $slot with @yield('content'). Because $slot is the definition of breeze layouts. I don't know why the author didn't mention use @yield('content') in the document.

Because this package doesn't say anything about using Laravel breeze.


Typo Error in the options and wrong option is set as correct answer in Data Analysis with Python Course - Numpy Operations

Describe your problem and how to reproduce it:
The question asked in this page is as follows:
_What is the value of a after you run the following code?
a = np.arange(5)
a + 20
[20, 21, 22, 24, 24]
[0, 1, 2, 3, 4]
[25, 26, 27, 28, 29]_
There is a typo error in the first option, I believe it should be [20, 21, 22, 23, 24]  which is the correct answer to the question asked.
Whereas the second option is set as the correct answer.
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/data-analysis-with-python/data-analysis-with-python-course/numpy-operations
Tell us about your browser and operating system:

Browser Name: Chrome
Browser Version:
Operating System: Windows

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

try it yourself in an editor, the value of a is between the options


custom build for Storm, is it ok?

While Installing Storm using these steps to custom build boost (see last section)
https://www.stormchecker.org/documentation/obtain-storm/dependencies.html
I have obtained not obvious result whether the installation succeeded or not:
If not, can you suggest a fix, please? Thanks.
	$ wget https://sourceforge.net/projects/boost/files/boost/1.63.0/boost_1_63_0.tar.gz
	$ tar -xzf boost_1_63_0.tar.gz
	$ cd boost_1_63_0
	$ ./bootstrap.sh
	Building Boost.Build engine with toolset gcc... tools/build/src/engine/bin.linuxx86_64/b2
	Detecting Python version... 3.8
	Detecting Python root... /home/matej/anaconda3/envs/py38
	Unicode/ICU support for Boost.Regex?... /usr
	Backing up existing Boost.Build configuration in project-config.jam.2
	Generating Boost.Build configuration in project-config.jam...

	Bootstrapping is done. To build, run:

	    ./b2

	To adjust configuration, edit 'project-config.jam'.
	Further information:

	   - Command line help:
	     ./b2 --help

	   - Getting started guide:
	     http://www.boost.org/more/getting_started/unix-variants.html

	   - Boost.Build documentation:
	     http://www.boost.org/build/doc/html/index.html
	$ ./bjam
	Performing configuration checks

	    - 32-bit                   : no  (cached)
	    - 64-bit                   : yes (cached)
	    - arm                      : no  (cached)
	    - mips1                    : no  (cached)
	    - power                    : no  (cached)
	    - sparc                    : no  (cached)
	    - x86                      : yes (cached)

	Building the Boost C++ Libraries.

	    - symlinks supported       : yes (cached)
	    - C++11 mutex              : yes (cached)
	    - lockfree boost::atomic_flag : yes (cached)
	    - Boost.Config Feature Check: cxx11_auto_declarations : yes (cached)
	    - Boost.Config Feature Check: cxx11_constexpr : yes (cached)
	    - Boost.Config Feature Check: cxx11_defaulted_functions : yes (cached)
	    - Boost.Config Feature Check: cxx11_final : yes (cached)
	    - Boost.Config Feature Check: cxx11_hdr_mutex : yes (cached)
	    - Boost.Config Feature Check: cxx11_hdr_tuple : yes (cached)
	    - Boost.Config Feature Check: cxx11_lambdas : yes (cached)
	    - Boost.Config Feature Check: cxx11_noexcept : yes (cached)
	    - Boost.Config Feature Check: cxx11_nullptr : yes (cached)
	    - Boost.Config Feature Check: cxx11_rvalue_references : yes (cached)
	    - Boost.Config Feature Check: cxx11_template_aliases : yes (cached)
	    - Boost.Config Feature Check: cxx11_thread_local : yes (cached)
	    - Boost.Config Feature Check: cxx11_variadic_templates : yes (cached)
	    - has_icu builds           : yes (cached)
	warning: Graph library does not contain MPI-based parallel components.
	note: to enable them, add "using mpi ;" to your user-config.jam
	    - zlib                     : yes (cached)
	    - bzip2                    : yes (cached)
	    - iconv (libc)             : yes (cached)
	    - icu                      : yes (cached)
	    - native-atomic-int32-supported : yes (cached)
	    - native-syslog-supported  : yes (cached)
	    - pthread-supports-robust-mutexes : yes (cached)
	    - compiler-supports-visibility : yes (cached)
	    - compiler-supports-ssse3  : yes (cached)
	    - compiler-supports-avx2   : yes (cached)
	    - gcc visibility           : yes (cached)
	    - long double support      : yes (cached)
	warning: skipping optional Message Passing Interface (MPI) library.
	note: to enable MPI support, add "using mpi ;" to user-config.jam.
	note: to suppress this message, pass "--without-mpi" to bjam.
	note: otherwise, you can safely ignore this message.
	    - zlib                     : yes (cached)
	    - bzip2                    : yes (cached)

	Component configuration:

	    - atomic                   : building
	    - chrono                   : building
	    - container                : building
	    - context                  : building
	    - coroutine                : building
	    - coroutine2               : building
	    - date_time                : building
	    - exception                : building
	    - fiber                    : building
	    - filesystem               : building
	    - graph                    : building
	    - graph_parallel           : building
	    - iostreams                : building
	    - locale                   : building
	    - log                      : building
	    - math                     : building
	    - metaparse                : building
	    - mpi                      : building
	    - program_options          : building
	    - python                   : building
	    - random                   : building
	    - regex                    : building
	    - serialization            : building
	    - signals                  : building
	    - system                   : building
	    - test                     : building
	    - thread                   : building
	    - timer                    : building
	    - type_erasure             : building
	    - wave                     : building

	...patience...
	...patience...
	...patience...
	...patience...
	...patience...
	...patience...
	...found 16129 targets...
	...updating 20 targets...
	gcc.compile.c++ bin.v2/libs/python/build/gcc-7/release/link-static/threading-multi/converter/builtin_converters.o
	libs/python/src/converter/builtin_converters.cpp: In function ‘void* boost::python::converter::{anonymous}::convert_to_cstring(PyObject*)’:
	libs/python/src/converter/builtin_converters.cpp:51:35: error: invalid conversion from ‘const void*’ to ‘void*’ [-fpermissive]
	       return PyUnicode_Check(obj) ? _PyUnicode_AsString(obj) : 0;

	    "g++"  -ftemplate-depth-128 -O3 -finline-functions -Wno-inline -Wall -pthread -m64  -DBOOST_ALL_NO_LIB=1 -DBOOST_PYTHON_SOURCE -DBOOST_PYTHON_STATIC_LIB -DNDEBUG  -I"." -I"/home/matej/anaconda3/envs/py38/include/python3.8" -c -o "bin.v2/libs/python/build/gcc-7/release/link-static/threading-multi/converter/builtin_converters.o" "libs/python/src/converter/builtin_converters.cpp"

	...failed gcc.compile.c++ bin.v2/libs/python/build/gcc-7/release/link-static/threading-multi/converter/builtin_converters.o...
	...skipped <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>libboost_python.a(clean) for lack of <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>converter/builtin_converters.o...
	...skipped <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>libboost_python.a for lack of <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>converter/builtin_converters.o...
	...skipped <pstage/lib>libboost_python.a for lack of <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>libboost_python.a...
	...skipped <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>libboost_python3.a(clean) for lack of <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>converter/builtin_converters.o...
	...skipped <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>libboost_python3.a for lack of <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>converter/builtin_converters.o...
	...skipped <pstage/lib>libboost_python3.a for lack of <pbin.v2/libs/python/build/gcc-7/release/link-static/threading-multi>libboost_python3.a...
	gcc.compile.c++ bin.v2/libs/python/build/gcc-7/release/threading-multi/converter/builtin_converters.o
	libs/python/src/converter/builtin_converters.cpp: In function ‘void* boost::python::converter::{anonymous}::convert_to_cstring(PyObject*)’:
	libs/python/src/converter/builtin_converters.cpp:51:35: error: invalid conversion from ‘const void*’ to ‘void*’ [-fpermissive]
	       return PyUnicode_Check(obj) ? _PyUnicode_AsString(obj) : 0;

	    "g++"  -ftemplate-depth-128 -O3 -finline-functions -Wno-inline -Wall -pthread -fPIC -m64  -DBOOST_ALL_NO_LIB=1 -DBOOST_PYTHON_SOURCE -DNDEBUG  -I"." -I"/home/matej/anaconda3/envs/py38/include/python3.8" -c -o "bin.v2/libs/python/build/gcc-7/release/threading-multi/converter/builtin_converters.o" "libs/python/src/converter/builtin_converters.cpp"

	...failed gcc.compile.c++ bin.v2/libs/python/build/gcc-7/release/threading-multi/converter/builtin_converters.o...
	...skipped <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_python.so.1.63.0 for lack of <pbin.v2/libs/python/build/gcc-7/release/threading-multi>converter/builtin_converters.o...
	...skipped <pstage/lib>libboost_python.so.1.63.0 for lack of <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_python.so.1.63.0...
	...skipped <pstage/lib>libboost_python.so for lack of <pstage/lib>libboost_python.so.1.63.0...
	...skipped <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_numpy.so.1.63.0 for lack of <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_python.so.1.63.0...
	...skipped <pstage/lib>libboost_numpy.so.1.63.0 for lack of <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_numpy.so.1.63.0...
	...skipped <pstage/lib>libboost_numpy.so for lack of <pstage/lib>libboost_numpy.so.1.63.0...
	...skipped <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_python3.so.1.63.0 for lack of <pbin.v2/libs/python/build/gcc-7/release/threading-multi>converter/builtin_converters.o...
	...skipped <pstage/lib>libboost_python3.so.1.63.0 for lack of <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_python3.so.1.63.0...
	...skipped <pstage/lib>libboost_python3.so for lack of <pstage/lib>libboost_python3.so.1.63.0...
	...skipped <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_numpy3.so.1.63.0 for lack of <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_python.so.1.63.0...
	...skipped <pstage/lib>libboost_numpy3.so.1.63.0 for lack of <pbin.v2/libs/python/build/gcc-7/release/threading-multi>libboost_numpy3.so.1.63.0...
	...skipped <pstage/lib>libboost_numpy3.so for lack of <pstage/lib>libboost_numpy3.so.1.63.0...
	...failed updating 2 targets...
	...skipped 18 targets...

probably the same result using gcc and g++ compliers version 9.3.0
(very very long output - sent upon a request)
probably the same result using gcc and g++ compliers version 6.5.0
(very very long output - sent upon a request)
...failed updating 2 targets...
...skipped 18 targets...
...updated 1180 targets...

Please note that 1.63.0 is a very old boost release (released in 2017) - we just (last week) released 1.75.0.


Editor: Cannot publish story 'immediately'

Bug Description

In the WordPress block editor, one can start creating a blog post, save regularly, and once you publish, it will use that moment's current date as the publish date.
In the story editor, once you start creating a story, that moment's date is set as the publish date. So even if you actually publish your story a week later, the week before will be the publish date.
This is wrong, and causes lots of stories to be published with the incorrect date.
Possible solution
I assume this can be easily fixed by not setting the publish date in the editor and keep it as null. So once you publish, WP will simply use the current date.
Expected Behaviour

Users should be able to publish "immediately".
Steps to Reproduce

Create a new story, save it
Publish it 10 minutes later
Notice the publish date is the date from 10 minutes earlier

Screenshots

"Publish Immediately" in Gutenberg:

Publish date in the story editor:

Additional Context

Plugin Version: 1.7.2
WordPress Version: 5.7
Operating System: macOS
Browser: Chrome

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Assigning to Prometheus but perhaps it's more of a WP thing.
cc @miina

So once you publish, WP will simply use the current date.

@swissspidy What if the user will mark the story temporarily a draft and then publishes it again? Should it always use the original publish date (unless explicitly changed by the user to a later date)? Thoughts?

So once you publish, WP will simply use the current date.

@swissspidy What if the user will mark the story temporarily a draft and then publishes it again? Should it always use the original publish date (unless explicitly changed by the user to a later date)? Thoughts?

Yes. Once the user publishes the story, the publish date will be set and left untouched until the user changes it manually.
Other scenarios can be compared to how it works in Gutenberg, as that is the experience our users are familiar with and expecting.

Hm, in this case, should we display "Immediately" or something similar instead of a specific date (unless it's already set), as in WP? Otherwise, displaying the date is not really accurate for a draft.

Or is this exactly what you've meant with this issue? :)

To consider when implementing: a difference in the editor vs in Gutenberg -- in the editor, the user can publish the post by changing the status to publish from the Status & visibility panel, too. Then the "Update" button needs to be clicked to save the changed status.

Verified in QA


Can't access partial_pipeline session variables when creating new user

I am using the SOCIAL_AUTH_PIPELINE to create a form that will allow a new user that is registering to create their own username. In the form I am creating, I would like to slugify their Facebook fullname for the form's deault, but I am unable to access the partial_pipeline variables from request.session during the registration process. Here is my pipeline:
SOCIAL_AUTH_PIPELINE = {
        'social_auth.backends.pipeline.social.social_auth_user',
        'social_auth.backends.pipeline.associate.associate_by_email',
        'social_auth.backends.pipeline.misc.save_status_to_session',
        'main.pipeline.redirect_to_socialsignup_form',
        'main.pipeline.username',
        'social_auth.backends.pipeline.user.create_user',
        'social_auth.backends.pipeline.social.associate_user',
        'social_auth.backends.pipeline.social.load_extra_data',
        'social_auth.backends.pipeline.user.update_user_details',
}

and here is the relavent portion of my view that is used during the 'main.pipeline.redirect_to_socialsignup_form', pipeline:
def social_signup(request):
    username = slugify(request.session.get("partial_pipeline")['kwargs']['details']['fullname'])

When the user is new, and encounters this view, I get this error:
-> username = slugify(request.session.get("partial_pipeline")['kwargs']['details']['fullname'])
(Pdb) username
*** NameError: name 'username' is not defined

But if the user has been through the signup process once, this works with no problem. It looks like I am trying to access the partial_pipeline variables before they have been set. Should I re-arrange my pipeline or is there a different way I should be trying to access this information?

I have done more troubleshooting. I added a pdb.set_trace() to ``social_auth.backends.pipeline.save_status_to_sessionand I have noticed that the pipeline does not get run when a new user registers for the first time. When the user is sent to the facebook authorization page, accepts the terms and then gets redirected to the site, thesave_status_to_session``` is not run. Maybe this is to be expected. Let me know your thoughts on this.

No, that's not correct, it should run always when the user is redirected back to your site from facebook, any more debugging info?

I am not sure where to put pdb.set_trace() to see what is going on. Can you recommend a section for me to drop it in at? Maybe something outside of the pipeline?

This seems to be working now, which is great. I looked at this diff. What was the issue? I was pretty sure that my users were not authenticated.

It's not a diff, just the line I've suggested to put a pdb.set_trace() call, so it works now?

Yes it is working now. I replaced the init.py in backends entirely with what you sent me. I am testing with a couple of other accounts and I will let you know if I run into trouble.

Which version of djangos-social-auth are you using?

0.6.6


[FEATURE] more informative building instructions (for Windows)

To build SkiaSharp on Windows:
this requires ~50GB for installation/setup
and an additional 44GB for building natives and bindings/views
building is recommended on EXTERNAL HARD-DRIVE if free space on internal hdd is small
install VS 2022
install specified https://github.com/mono/SkiaSharp/wiki/Building-SkiaSharp for 2022
NOTE: MSVC v142 - VS 2019 C++ x86/64 build tools are required for exec_script("//gn/highest_version_dir.py", [ "$win_vc/Tools/MSVC", "$win_vcvars_version[0-9]{1}\.[0-9]{5}" ], "trim string") to work in skia build script
~25GB for components installation
Windows 10 SDK (10.0.10240) can be obtained from https://go.microsoft.com/fwlink/p/?LinkId=619296
Windows 10 SDK (10.0.16299.91) can be obtained from https://go.microsoft.com/fwlink/p/?linkid=864422
Android NDK is not available in VS component selection
install Android NDK 21 from https://github.com/android/ndk/wiki/Unsupported-Downloads#r21e
Android NDK 22 restructures the following root paths, which are not yet adapted to this build system
   * platforms
   * sources/cxx-stl
   * sysroot
   * toolchains (with the exception of toolchains/llvm)

   In general this change should only affect build system maintainers, or those
   using build systems that are not up to date. ndk-build and the CMake
   toolchain users are unaffected, and neither are
   `make_standalone_toolchain.py` users (though that script has been unnecessary
   since r19).

go to System > Advanced System Settings > Environment Variables
add the following
name:  ANDROID_SDK_ROOT
value: C:\Program Files (x86)\Android\android-sdk

the above is where VS installs the sdk to
name:  ANDROID_NDK_ROOT
value: C:\Users\small\Downloads\android-ndk-r21

(replace "small" with your username)
restart your machine to apply the above changes to system environmental variables
open and replace ./scripts/install-android-platform.ps1 with the following content
Param(
    [string] $API
)

$ErrorActionPreference = 'Stop'

$sdk = "$env:ANDROID_SDK_ROOT"

$apiPath = "$sdk/platforms/android-$API/android.jar"
if (Test-Path $apiPath) {
    Write-Host "Android API level $API was already installed."
    exit 0
}

$latest = "$sdk/cmdline-tools/latest"
if (-not (Test-Path $latest)) {
    $versions = Get-ChildItem ("$sdk/cmdline-tools")
    $latest = "$sdk/cmdline-tools/" + ($versions | Select-Object -Last 1)[0]
}

if (-not $IsMacOS -and -not $IsLinux) {
    $ext = ".bat"
}

$sdkmanager = "$latest/bin/sdkmanager$ext"

Set-Content -Value "y" -Path "yes.txt"
try {
    if ($IsMacOS -or $IsLinux) {
        sh -c "`"$sdkmanager`" `"platforms\;android-$API`" < yes.txt"
    } else {
        cmd /c "`"$sdkmanager`" `"platforms;android-$API`" < yes.txt"
    }
} finally {
    Remove-Item "yes.txt"
}

exit $LASTEXITCODE

next execute the following in an admin powershell
./scripts/install-android-platform.ps1 29

execute the following in non-admin powershell
# note you will need to execute ./scripts/install-tizen.ps1
# multiple times due to bugs in waiting for process to finish
./scripts/install-tizen.ps1 # install tizen studio
# dont execute the next line until the above is done installing tizen studio

# note you will need to execute ./scripts/install-tizen.ps1
# multiple times due to bugs in waiting for process to finish
./scripts/install-tizen.ps1 # install packages from tizen studio package manager
# dont execute the next line until the above is done installing packages from tizen studio package manager

# note you will need to execute ./scripts/install-tizen.ps1
# multiple times due to bugs in waiting for process to finish
./scripts/install-tizen.ps1 # patch llvm in tizen
./scripts/install-llvm.ps1 # windows llvm
./scripts/install-gtk.ps1
./scrips/install-maui.ps1
./scrips/install-mono.ps1
dotnet workload install android ios tvos macos maccatalyst wasm-tools maui --source https://api.nuget.org/v3/index.json --source https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet6/nuget/v3/index.json --source https://pkgs.dev.azure.com/dnceng/public/_packaging/darc-pub-dotnet-runtime-bd261ea4/nuget/v3/index.json --source https://pkgs.dev.azure.com/dnceng/public/_packaging/darc-pub-dotnet-emsdk-52e9452f-3/nuget/v3/index.json

add the following to cake/msbuild.cake
find
        if (!string.IsNullOrEmpty(MSBUILD_EXE)) {
            c.ToolPath = MSBUILD_EXE;
        } else if (IsRunningOnWindows() && !string.IsNullOrEmpty(VS_INSTALL)) {
            c.ToolPath = ((DirectoryPath)VS_INSTALL).CombineWithFilePath("MSBuild/Current/Bin/MSBuild.exe");
        }

and replace it with
        if (!string.IsNullOrEmpty(MSBUILD_EXE)) {
            c.ToolPath = MSBUILD_EXE;
        } else if (IsRunningOnWindows()) {
            if (!string.IsNullOrEmpty(VS_INSTALL)) {
                c.ToolPath = ((DirectoryPath)VS_INSTALL).CombineWithFilePath("MSBuild/Current/Bin/MSBuild.exe");
            } else {
                // check for msbuild 17, this is required to build .net 6 projects (non-natives)
                // VS 2022 is in preview
                // check VS 2022 Community when it comes out of preview
                bool vs2022Cexists = DirectoryExists("C:/Program Files/Microsoft Visual Studio/2022/Community");
                bool vs2022Pexists = DirectoryExists("C:/Program Files/Microsoft Visual Studio/2022/Preview");
                if (!vs2022Pexists && !vs2022Cexists) {
                    throw new Exception("Visual Studio 2022 Preview or Community is required");
                }
                if (vs2022Pexists) {
                    c.ToolPath = "C:/Program Files/Microsoft Visual Studio/2022/Preview/MSBuild/Current/Bin/MSBuild.exe";
                }
                if (vs2022Cexists) {
                    c.ToolPath = "C:/Program Files/Microsoft Visual Studio/2022/Community/MSBuild/Current/Bin/MSBuild.exe";
                }
            }
        }

next, we can begin building
we will assume K:\ is location of SkiaSharp git repo and the repo has been cloned into SkiaSharp
first we need to grab all external libs needed to pack the nuget
dotnet cake --target=externals-download

any native builds will be built over the top of these
build/windows/skia.so -> out/native/windows/skia.so # overwrites downloaded native/windows/skia.so

when building on a different platform the above will need to be done to retain changes to native libs
Windows natives/windows/skia.so -> K:\skia.so
OSX download natives
OSX K:\skia.so -> natives/windows/skia.so // for windows

next we build and test everything
cd K:\SkiaSharp\ ; dotnet tool restore ; dotnet cake --target=everything --buildall=true

i feel like this should be pinned somewhere?


fix {gid} encode format

fix Retrieve SigRL' Parameters {gid} format.
according to api documentation, {gid} – Base 16-encoded representation of the EPID group ID provided by the
platform, encoded as a Big Endian integer.
https://github.com/litentry/litentry-parachain/pull/1042

Hi, thanks for the PR. A quick question here, did this fix your remote attestation, which was broken before?
It is interesting because it was never a problem with us, although maybe https://github.com/integritee-network/worker/pull/971 broke something. @Niederb have you tested that remote attestation worked after your refactoring?

Yes, we did test it and the refactoring was more about the overall architecture. I should not have affected little details.
I did a quick test and the following code gives me the same output:
	let val: u32 = 123456789;
	let gid = hex::encode(val.to_be_bytes());
	println!("Old: {:08x}", val);
	println!("New: {}", gid);

Output:
Old: 075bcd15
New: 075bcd15

@zTgx Can you test in your environment? Seems really strange to me if we have different results

Yes, we did test it and the refactoring was more about the overall architecture. I should not have affected little details. I did a quick test and the following code gives me the same output:
	let val: u32 = 123456789;
	let gid = hex::encode(val.to_be_bytes());
	println!("Old: {:08x}", val);
	println!("New: {}", gid);

Output:
Old: 075bcd15
New: 075bcd15

@zTgx Can you test in your environment? Seems really strange to me if we have different results

I tested it again, we have the same results.
Sorry, I made a mistake, it's not a bug, you can close it.

Nice, thanks for the forensic analysis lads.


Feature Request: Copy path and goto from top file path at top of page

You can right click a file from the Explorer frame and copy the file path or jump to it in Windows Explorer but you can not do the same from the file path that is at the top of each page.

from the file path that is at the top of each page

Related to #56494

Forwarding to @jrieken since this seems like a feature request for breadcrumbs

This is a duplicate. Unfortunately, I don't recall the exact issue number nor its title :flushed: Anyways, searching for issues with the same labels will likely help you :point_right:
https://github.com/microsoft/vscode/issues?utf8=✓&q=is%3Aopen+label%3Abreadcrumbs


S3: All file sizes exceed upload limit

This is a duplicate of #386, but the issue was closed, so I decided to open another one.
I copied the code from the AWS SDK doc examples here and get the following error when trying to upload a 128KB file as well an empty file:

PutObject error: EntityTooLarge Unable to parse ExceptionName: EntityTooLarge Message: Your proposed upload exceeds the maximum allowed size

The feedback from the other issue says:

This line is not doing what you think it is doing. The first argument of MakeShared is a memory allocation tag. The second argument on is what get's passed to the constructor. Are you intentionally passing you filename as the memory allocation tag?

I couldn't find any documentation about the memory allocation tag and would appreciate any feedback.
I also couldn't find any example code for the transfer manager and would appreciate any pointers about how to use that as well.
Thanks!

The memory allocation tag is just an arbitrary string so that you can classify or track your allocations.  Make it anything you want.
You can look at the tests in aws-cpp-sdk-transfer-tests for examples of how to use the transfer SDK.

Thanks, Bret. Still would love to get some feedback on how to avoid the PutObjectError.

My best guess is the PutObject error likely comes from the fact that either the stream was invalid or did not have a binary flag set (if the file was binary).  Can you step through the http client request stream upload logic and check the state of the stream?

Are you still having problems with this? Please reopen if you still have problems.


Added PHP 7.3 support

Updated Travis CI configuration to run all tests on PHP 7.3 with lowest/locked/latest dependencies

To fix the build, you might want to look into this:
http://php.net/manual/en/control-structures.continue.php
php/php-src#3364
https://wiki.php.net/rfc/continue_on_switch_deprecation

@xtreamwayz
The problem is only with lowest dependencies and problem is already fixed in zend-stdlib 3.2.1.
But here, zend-stdlib is required implicitly:
$ composer why zendframework/zend-stdlib

zendframework/zend-expressive-fastroute  3.0.2   requires  zendframework/zend-stdlib (^2.0 || ^3.1)
zendframework/zend-http                  2.8.2   requires  zendframework/zend-stdlib (^3.1 || ^2.7.7)
zendframework/zend-router                3.2.0   requires  zendframework/zend-stdlib (^2.7.7 || ^3.1)
zendframework/zend-servicemanager        3.3.2   requires  zendframework/zend-stdlib (^3.1)
zendframework/zend-validator             2.10.2  requires  zendframework/zend-stdlib (^2.7.6 || ^3.1)

Now I think the best solution will be to add conflict with zend-stdlib < 3.2.1 into composer.json and I am going to do in a sec :)

I disagree with adding zend-stdlib to conflicts.

@Xerkus

I disagree with adding zend-stdlib to conflicts.

Why?  Add it as explicit dependency?

Sorry, accidentally sent the message.
@webimpress I need to take a look into this. I'll get to it in a bit after I'll make a release of zend-file

There is no clear solution here.
One option is bump minimum stdlib version in dependencies and then bump minimum on those dependencies in turn.
Another option is to declare conflict.
I don't see what else can be done here. Proper solution would have been to specify upper minor php versions we support instead of open range. No going back on that one tho.
I am favouring first one slightly more as it allows us to avoid conflict on package we don't even use here.
I asked @Ocramius and he is OK with declaring conflict here.


fix: export K8sRequest, add Serialize

Please rewrite the poetry lock with $> poetry lock (you can remove the old poetry.lock beforehand) in the /json directory

@Schille Done. :)


OpenGL detect the version as 1.1

Software Versions

Python: 3.9
OS: windows 10
Kivy: 2.0.0
Kivy installation method: pip

i dont know how to create an issue properly, but i hope there is someone who knows exactly my problem. also im sorry if my english is bad.
so, when i run my app, in the command line show me that my opengl version is 1.1, but yesterday thats all are fine, and it show me that my opengl version is above 2. i ran the code in same machine but in the different days.
Expected behavior
i expected my code will run properly as before, before this error exist.
Logs
so, this is the log on 4 march:
[INFO   ] Logger: Record log in C:\Users\Thosiba\.kivy\logs\kivy_22-03-04_14.txt
[INFO   ] deps: Successfully imported "kivy_deps.angle" 0.3.0
[INFO   ] deps: Successfully imported "kivy_deps.glew" 0.3.0
[INFO   ] deps: Successfully imported "kivy_deps.sdl2" 0.3.1
[INFO   ] Kivy: v2.0.0
[INFO   ] Kivy: Installed at "C:\py_venv\kivy_venv\lib\site-packages\kivy\__init__.py"
[INFO   ] Python: v3.9.7 (tags/v3.9.7:1016ef3, Aug 30 2021, 20:19:38) [MSC v.1929 64 bit (AMD64)]
[INFO   ] Python: Interpreter at "C:\py_venv\kivy_venv\Scripts\python.exe"
[INFO   ] Factory: 186 symbols loaded
[INFO   ] Image: Providers: img_tex, img_dds, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] Window: Provider: sdl2
[INFO   ] GL: Using the "OpenGL" graphics system
[INFO   ] GL: GLEW initialization succeeded
[INFO   ] GL: Backend used <glew>
[INFO   ] GL: OpenGL version <b'4.0.0 - Build <IP_ADDRESS>25'>
[INFO   ] GL: OpenGL vendor <b'Intel'>
[INFO   ] GL: OpenGL renderer <b'Intel(R) HD Graphics'>
[INFO   ] GL: OpenGL parsed version: 4, 0
[INFO   ] GL: Shading version <b'4.00 - Build <IP_ADDRESS>25'>
[INFO   ] GL: Texture max size <16384>
[INFO   ] GL: Texture max units <16>
[INFO   ] Window: auto add sdl2 input provider
[INFO   ] Window: virtual keyboard not allowed, single mode, not docked
[INFO   ] Text: Provider: sdl2
[WARNING] Both Window.minimum_width and Window.minimum_height must be bigger than 0 for the size restriction to take effect.
[INFO   ] GL: NPOT texture support is available
[INFO   ] Base: Start application main loop
[INFO   ] Base: Leaving application in progress...

and this is kivy log on 5 march:
[INFO   ] Logger: Record log in C:\Users\Thosiba\.kivy\logs\kivy_22-03-05_0.txt
[INFO   ] deps: Successfully imported "kivy_deps.angle" 0.3.0
[INFO   ] deps: Successfully imported "kivy_deps.glew" 0.3.0
[INFO   ] deps: Successfully imported "kivy_deps.sdl2" 0.3.1
[INFO   ] Kivy: v2.0.0
[INFO   ] Kivy: Installed at "C:\py_venv\kivy_venv\lib\site-packages\kivy\__init__.py"
[INFO   ] Python: v3.9.7 (tags/v3.9.7:1016ef3, Aug 30 2021, 20:19:38) [MSC v.1929 64 bit (AMD64)]
[INFO   ] Python: Interpreter at "C:\py_venv\kivy_venv\Scripts\python.exe"
[INFO   ] Factory: 186 symbols loaded
[INFO   ] Image: Providers: img_tex, img_dds, img_sdl2, img_pil (img_ffpyplayer ignored)
[INFO   ] Window: Provider: sdl2
[INFO   ] GL: Using the "OpenGL" graphics system
[INFO   ] GL: GLEW initialization succeeded
[INFO   ] GL: No framebuffers extension is supported
[INFO   ] GL: Backend used <glew>
[INFO   ] GL: OpenGL version <b'1.1.0'>
[INFO   ] GL: OpenGL vendor <b'Microsoft Corporation'>
[INFO   ] GL: OpenGL renderer <b'GDI Generic'>
[INFO   ] GL: OpenGL parsed version: 1, 1
[CRITICAL] GL: Minimum required OpenGL version (2.0) NOT found!

OpenGL version detected: 1.1

Version: b'1.1.0'
Vendor: b'Microsoft Corporation'
Renderer: b'GDI Generic'

Try upgrading your graphics drivers and/or your graphics hardware in case of problems.

The application will leave now.

Additional context
i tried to change the multisample to 0, and yeah its work. but, when i do that, the log showing that opengl version is above 2.0. but when i change it back to 2, i got opengl version is 1.1.

Looks like the driver is changing between the generic one and the Intel specific.
The issue still persists with Kivy 2.1.0?

Looks like the driver is changing between the generic one and the Intel specific. Does the issue still persist with Kivy 2.1.0?

i didnot change the driver. i got 2 item of driver list shown up when i check driver update in device manager. that are "intel(r) HD graphic" and "microsoft basic display adapter", even i change the driver it still doesnot work.
by the way, how do i get 2.1.0 version of kivy? please tell me

also, since that error exist. when i set multisamples to 2, my terminal shows that my OpenGL vendor is "microsoft corporation". but, if i set the multisamples to 0, which is how my code works, my terminal shows that my OpenGL vendor is "intel hd graphic"

If you pip installed Kivy,
Looks like a duplicate of https://github.com/kivy/kivy/issues/3576
Can you post your PC configuration

If you pip installed Kivy, pip install --upgrade kivy updates it to 2.1.0
BTW, looks like a duplicate of https://github.com/kivy/kivy/issues/3576
Can you post your PC configuration over there so we can close this issue and keep only one?

by the way, I just found the solution recently. I think turning off anti aliasing could cause this error to appear. and i tried to turn the aliasing on, and yah it works.

I went to my file manager and located my present working directory, opened up the venv folder and then opened up the share folder. I pasted the file I downloaded from the link below (extract first to get the opengl32.dll) into all the bin folder (for angle, gstreamer e.t.c). it also works when you try to package your application with pyinstaller
https://www.dll-files.com/download/b8db21516b088afd9b0242741f435c71/opengl32.dll.html?c=VHRPVEJkcGJ2ZGx4V0JOdlZoYlhSZz09


DEMO Bug: moving slider on option 2 causes option 1 to still fire

This is a weird one, but moving the handle on option 2 causes both option 1 and option 2 code to run. Weird encapsulation issues with demo.

Assuming this is fixed in the new demo


Add breadcrumb navigation on FAQ article detail page

Acceptance criteria

when a user is on FAQ article detail screen, he can click on any breadcrumb displayed above article title to navigate back to that specific category

Zeplin: https://zpl.io/aM48YJr or in app zpl://screen?sid=5aa98f7f3820cb498e96a09a&pid=5a8e8992554d385c76ad5aeb

👍


Issue with chroma on vercel production

"chromadb": "1.5.2", (in this version of chroma there is no warnings about module not found like https://github.com/langchain-ai/langchainjs/issues/346)
"langchain": "0.0.198",
"next": "13.5.4",
When I'm querying chroma on localhost it there is no issue, everything works like a charm but on production build on vercel it does not work for some reason.
Also, code for making a collection works just fine on both builds and I am able to find the created collection on chroma directly from the docs.
There is no straight error, just not working with status 500
Code for querying (working only on localhost):
export const runtime = 'edge';

export async function POST(req: Request){
  try {
    const body = await req.json()
    const messages = body.messages ?? [];
    const question = messages[messages.length - 1].content
    const { history, collectionId } = body

    const sanitizedQuestion = question.trim().replaceAll('\n', ' ');

    const { stream, handlers } = LangChainStream();

    const vectorStore = await Chroma.fromExistingCollection(
      new OpenAIEmbeddings(),
      {
        url: env.CHROMA_DB_SERVER,
        collectionName: collectionId,
      },
    );

    const chain = makeChain(vectorStore);

    chain.call({
      question: sanitizedQuestion,
      chat_history: history
    }, [handlers]);

    return new StreamingTextResponse(stream);
  } catch (error) {
    return NextResponse.json({message: "Internal error"}, {status: 500})
  }
}

Code for creating the collection (working both on localhost and prod):
export async function POST(req: Request){
  try {
    const body = await req.formData()
    const file:any = body.get('file')
    const fileName:string = file.name
    const userId:any = body.get('userId')

    const collectionName = await prisma.collection.create({
      data: {
        fileName: fileName,
        userId: userId,
      }
    })

    const loader = new WebPDFLoader(file);
    const docs = await loader.load();

    const splitter = new RecursiveCharacterTextSplitter({
      chunkSize: 2000,
      chunkOverlap: 500,
    });

    const documents = await splitter.splitDocuments(docs);

    const vectorStore = await Chroma.fromDocuments(
      documents,
      new OpenAIEmbeddings(),
      {
        url: env.CHROMA_DB_SERVER,
        collectionName: collectionName.id
      }
    );

    return NextResponse.json({message: "Embbed successfull!", collectionName: collectionName.id}, {status:200})
  } catch (error) {
    return NextResponse.json({message: "Internal error"}, {status: 500})
  }
}

What's the value of error here?
  } catch (error) {
    return NextResponse.json({message: "Internal error"}, {status: 500})
  }

This is the error:
Error: Chroma getOrCreateCollection error: Error: Unexpected token 'D', "Direct IP "... is not valid JSON

CC @jeffchuber  do you have any idea?

@andreasjoensson are you using latest 1.6.1 from https://www.npmjs.com/package/chromadb?

Ah probably not supported on our end - will bump later and cut another release.

@jeffchuber yes i am using that version, i will try to downgrade and use another version in the meantime. Thanks for the quick support guys :)

Yeah, appreciate it. 🥇
@andreasjoensson please share if you are able to get it working with downgrading.

@moreno1123 Will do i am right now trying to find which Langchain and ChromaDB versions matches, because when i use a older version of ChromaDB with Langchain i get Error occurred: Error: Please install chromadb as a dependency with, e.g. npm install -S chromadb``
Will reach back if i find a solution :)

I can try later today as well - just to clarify it sounds like it's an issue with moving from a local deployment to a new cloud hosted version?

@jacoblee93 I used hosted Chroma on AWS EC2 in both cases, so it's not that...

@jacoblee93 Hey i'm also using a hosted Chroma on Render in my case, i tried to downgrade but could not seem to make it work. Everything has been working completely fine until yesterday, where suddenly my chatbot was not able to answer any questions based on the vector store and seemed to has lost all its knowledge.
When checking the logs my ChromaDB returns 200 both on create and queries, and i have not changed anything so i'm a bit confused as to how this would happen.

I can provide you with my code, if that is of any help.

@jacoblee93 i figured it out it was a error on my end, i'm really sorry. I had accidentally made a change somewhere, so that the wrong collectionID was used. But thanks for the quick support appreaciate it, have a great weekend :)

It's not sorted on my end though....

@moreno1123 - Please see my comment here on how I worked around the missing chroma issue. Hope it helps - https://github.com/langchain-ai/langchainjs/issues/2637#issuecomment-2552735141


MAIN - B 19813 - Customer time validation - HHG, NTS, NTSR

B-19813
INT PR
Summary
MAIN
This PR adds date validation for HHG, NTS and NTSR for preferred pickup and/or delivery date selection for the shipment onboarding process. An inline alert message will appear upon date selection from date picker. An alert message will appear on page load and on date selection. Message will be customized to respective date label. Alert will notify user if date selected is on weekend and/or holiday(federal holidays). For now implementation will assume date context is in the United States. For example, July 4th is a holiday and Sat/Sunday are considered weekend days.
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Have the Agility acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test
Alerts will only display if date is on a weekend and/or holday.

Login as customer
Add shipment types: HHG, NTS, NTSR
For each shipment verify alert messages are displayed when holidays and weekend dates are selected for preferred pickup and delivery dates during onboarding.
Verify alerts are displayed on shipment edit.
Verify alert messages are for pickup or delivery.

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Backend

[ ] Code follows the guidelines for Logging.
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Letting @deandreJones chime in on this one.
I know the change is small, but this PR has been branched off of its downstream PR which makes these dependent on each other.
IMO this is not ideal coding practices and could cause issues down the way - no matter how small the changes may seem.
Once Dre has chimed in, I'll check back.

Agree- It is not good practice to have 2 branches dependent on each other- maybe should have been combined-
circular dependencies make things messy
I agree it may not be convenient to go back and make changes to INT, but it keeps other from having conflicts, if those branches aren't merged together

Letting @deandreJones chime in on this one.
I know the change is small, but this PR has been branched off of its downstream PR which makes these dependent on each other.
IMO this is not ideal coding practices and could cause issues down the way - no matter how small the changes may seem.
Once Dre has chimed in, I'll check back.

Agree- It is not good practice to have 2 branches dependent on each other- maybe should have been combined- circular dependencies make things messy I agree it may not be convenient to go back and make changes to INT, but it keeps other from having conflicts, if those branches aren't merged together

Agree it should have gone in with the INT branch with this backlog but now both INT and Main have the change in question I am going to go ahead and merge this into main.

This was merged in with another PR so closing this as its no longer needed.


revert "Flip RTL text before sending it #184"

seems like it's been fixed in kodi while ago and we should not reverse the input string anymore.
@SyncedSynapse

If minimal version of Kodi that Kore supports has the original issue fixed, then simply removing the code is enough. Otherwise, you need to guard that with a flag that is set once on connection estabilishment.

@basilgello reversing the text has never been a correct solution. imagine a situation in which our text is a mixture of rtl and ltr words.

OK, just checking as I was not aware of the problem (and its "fix")

No idea about what the correct behaviour should be.
@amiremohamadi  Do you have a link to the Kodi PR that fixed the issue?
@royiarchy Do you have any input into this.
If the issue was fixed in a specific version of Kodi, what basilgello was referring to was keeping the code, but guarding it with something like
'''
if (! hostManager.getHostInfo().isLeiaOrLater) {
// Removed code to reverse text
}
'''
Not sure if necessary

@SyncedSynapse I guess it's solved in this commit.
IMO keeping the previous code (reversing the whole text) for the older versions is unnecessary as it will still mess up mixed RTL/LTR texts (even in the older versions of kodi).


Flexbug1 - Minimum content sizing of flex items not honored

https://github.com/philipwalton/flexbugs/blob/master/README.md#1-minimum-content-sizing-of-flex-items-not-honored

This can't be fixed with this tool because there's not enough metadata to take action.
Bug
.FlexItem {
  background: hsla(0,0%,0%,.1);
  margin: 1em;
  padding: 1em;
}

Workaround
.FlexItem {
  flex-shrink: 0;
  background: hsla(0,0%,0%,.1);
  margin: 1em;
  padding: 1em;
}

Hey wasn't sure if I should comment here or open a new issue but I have attempted to fix this bug by checking for display: flex and creating a new rule with parentSelector > * { flex-shrink:0; }.
Was curious what you think of this solution? I can push up my current work if you'd like to take a look.

so, for every 'display: flex' class, add this new rule?


OSFUSE-47 - general Karaf readiness probe for Karaf template and quic…

…kstarts.
Lets discover if the karaf deployment has health checks enabled and add them to the k8s config.

add to whitelist

test this please

lgtm

lgtm [merge]

[merge]

lgtm [merge]


No need for this over arching issue. If there is a piece of code that needs to be refactored, an issue can be made for that specific case.


[TypeChecker] NFC: Make test-case for rdar://problem/33688063 more co…

…mplicated
SE-0213 improved cases like that but we still have problem with operator
overloads, so just need to make this a bit more complicated to reproduce again.
Resolves: rdar://problem/42304000

@swift-ci please smoke test

@swift-ci please smoke test Linux platform


Executor info is wrong in Processor tab

We show all executor of the app, instead of executor used by this processor.

The restapi does not provide a mapping of processors and executors.

now, the mapping is provided.

fixed in ui v2


Embedded quotes not allowed in Java WebStart, therefore no PDF generation

Hey there,
I successfully implemented documents4j to convert DOCX files to PDF running the server standalone version with the LocalConverter.
IConverter converter = LocalConverter.builder().baseFolder(new File("D:/somefolder")).build();
When I start my jar with the normal way with JRE it works like a charm. But when I start it with Java WebStart (all-permissions), it generates only empty PDF files, even when I host the files on the same machine (localhost).
I started the vbs script with proper arguments (convert.vbs) manually and it works fine. MS Office is installed correctly. In the WebStart console it gives the following error:
Cannot run program "cmd" (in directory "D:\somefolder"): Malformed argument has embedded quote: D:\somefolder\word_start2012150060.vbs"
In my folder there are scripts generated. But the random number does not match, so it cannot find the file to execute. Why the programm generates a file with a random number, but tries to load that file with another random number and therefore cant find that generated file? How can i solve it?
Greetings

That's strange, it shold materialize it to a folder and remember the materialized name. You would need to debug documents4j to see what happens when the name is written and what name is remembered. I wonder if this is WebStart specific or a general bug, maybe a race.

Hi,
so I figured out it has nothing to do with the random number. I got confused, because the system deletes the word_start script after it failed to execute it.
In com.documents4j.conversion.AbstractExternalConverter.runNoArgumentScript() it tries to execute the script with code:
makePresetProcessExecutor()                     .command("cmd", "/S", "/C", doubleQuote(script.getAbsolutePath()))                     .execute().getExitValue();
When i start it in the normal JRE and with WebStart the console shows:
17:03:48.298 [AWT-EventQueue-0] DEBUG org.zeroturnaround.exec.ProcessExecutor - Executing [cmd, /S, /C, ""E:\somefolder\word_start873416445.vbs""] in E:\somefolder.
Ok thats fine. But in WebStart I got following error:
Cannot run program "cmd" (in directory "E:\somefolder"): Malformed argument has embedded quote: "E:\somefolder\word_start2116261157.vbs" 17:05:54.398 [AWT-EventQueue-2] ERROR com.documents4j.conversion.msoffice.MicrosoftWordBridge - Unable to run script: E:\somefolder\word_start2116261157.vbs java.io.IOException: Could not execute [cmd, /S, /C, ""E:\somefolder\word_start2116261157.vbs""] in E:\somefolder.
So I think that error has something to do with the doubleQuotes. But why WebStart behaves different that the normal Run with the JRE. I tried to just left out the double quotes but than the conversion doesnt work at all.
"Conversion failed for an unknown reason"
What can I actually do now? Any hints?
Thanks

When I leave out all quotes in the last argument of .command() it works!!!! But this only works for paths without a whitespace. I guess WebStart dont allow double quotes. I tried to mix ' and " but no success.
Double Quotes seem(s) to be an issue:
https://www.rapid7.com/db/modules/exploit/windows/browser/java_ws_double_quote/
But dont know how to handle this right now. Is there another way to run this script without the double quotes?
Regards

I can add a parameter to avoid them. Weird that this is platform dependant.

I have to admit that I dont know how to do this. And what you mean by "add a check on system property in doubleQuote"?

Ok just to clear it up. The problem is not the double quotes like "test". Actually it is the embedded quote like ""test"". I falsly named it double quotes, i guess. So again: the problem in WebStart are embedded quotes.

You'd need to adjust this function and then build documents4j: https://github.com/documents4j/documents4j/blob/218a962e3fa71c0dad239e3867dbd9116961e84e/documents4j-util-transformer-process/src/main/java/com/documents4j/conversion/AbstractExternalConverter.java#L42
I am not sure why this is not working. Via the cmd indirection, I experienced that the double quotes are always required. I would not even understand how WebStart changes the command dispatching.

I also had to remove the quotes in:
https://github.com/documents4j/documents4j/blob/218a962e3fa71c0dad239e3867dbd9116961e84e/documents4j-util-transformer-process/src/main/java/com/documents4j/conversion/AbstractExternalConverter.java#L28
It is a bit of a dirty workaround because now it only works, when the base directory given in LocalConverter.builder().baseFolder() has no whitespace in its path.
I tried all different types of quotation to avoid this behaviour, but no success. I also thought about a different way of running the script with arguments from java. But the return value will also be used in the code and it would be a really big workload to change all this.
I changed the title for better search results.
Thanks for your help

It's weird that this should be required as it breaks the converter when folders with spaces are used what, unfortunately, is a rather common problem on Windows systems. Since this seems to be a requirement, you are welcome to add a PR to documents4j where this can be enabled using a system property.

you are welcome to add a PR to documents4j where this can be enabled using a system property
I dont get it. What system property do you mean?

You'd need to introduce one and check it in the changed code and remove the double quote, if set.


Update to Chrome Browser breaks EPUB uploads

Apparently, the latest update to the Chrome Browser (44.0.2403.107m) has broken EPUB uploads.  I have verified this on my wife's Windows 8.1 Lenovo laptop.  Tried to upload "Accessible EPUB" and it hangs almost immediate with the message "Examining EPUB..."  Browser window has to be closed.
Now verified on OSX as well.  Same version of the browsr.

Additional info.  Happens with the latest build in develop as well.  Verified on OSX.

Console dump when error occurs:
 readium-js-viewer_all_CHROMEAPP.js:11577 RequireJS ... cfi_instructions
 readium-js-viewer_all_CHROMEAPP.js:12014 RequireJS ... cfi_interpreter
 readium-js-viewer_all_CHROMEAPP.js:12520 RequireJS ... cfi_generator
 readium-js-viewer_all_CHROMEAPP.js:12667 RequireJS ... cfi_API
 requirejs-config.js:25 chrome-extension://eafhhobkochaioojdidffhllaimjfdkk/index.html
 requirejs-config.js:29 chrome-extension://eafhhobkochaioojdidffhllaimjfdkk
 readium-js-viewer_all_CHROMEAPP.js:26950 'Range.detach' is now a no-op, as per DOM      (http://dom.spec.whatwg.org/#dom-range-detach).
 readium-js-viewer_all_CHROMEAPP.js:29664 Discontiguous selection is not supported.(anonymous function) @ readium-js-viewer_all_CHROMEAPP.js:29664
 readium-js-viewer_all_CHROMEAPP.js:12654 #######################################
 readium-js-viewer_all_CHROMEAPP.js:21901 Globals...
 readium-js-viewer_all_CHROMEAPP.js:21907 ReadiumSDK set.
 readium-js-viewer_all_CHROMEAPP.js:55924 'KeyboardEvent.keyLocation' is deprecated. Please use 'KeyboardEvent.location' instead.
 requirejs-config.js:25chrome-extension://eafhhobkochaioojdidffhllaimjfdkk/scripts/readium-js-viewer_CHROMEAPP-WORKER.js
 requirejs-config.js:29 chrome-extension://eafhhobkochaioojdidffhllaimjfdkk
 ZipFileLoader.js:29 Uncaught TypeError: Cannot set property 'mimetype' of undefined(anonymous function) @ ZipFileLoader.js:29(anonymous function) @ zip.js:670reader.onload @ zip.js:180
 WorkerProxy.js:96 [ErrorEvent]0: ErrorEventcallee: ()length: 1Symbol(Symbol.iterator):      ArrayValues()__proto__: Objectworker.onerror @ WorkerProxy.js:96

A lot of unfamiliar logging happening in the console. Did we update the chrome extension recently?

I do not know.  Most likely Juan or Daniel.  AFAIK, we did not modify or update the Chrome Extension per se.  But I don't think that has anything to do with this.  Users are reporting the problem in the production version (2.18.1) and I am seeing it in the latest (20) code in the develop branch.  Appears to be some exception thrown from the unzipping process.  But that area is a foreign country to me... :-)
BTW, the new release of Chrome came out on Friday.
http://googlechromereleases.blogspot.com/

from @ryanackley
So it seems that 'self' references the global scope when you are inside a worker. The equivalent of 'window'.  I didn't realize this. It must have been made read-only in the latest version. I changed the name to avoid any future confusion.


Running tests errors out when using --coverage

Do you want to request a feature or report a bug?
bug
What is the current behavior?
Running the tests normally works, but if i add the coverage flag, it fails:
babel-plugin-jest-hoist: The module factory of `jest.mock()` is not allowed to reference any out-of-scope variables.

    Note: This is a precaution to guard against uninitialized mock variables. If it is ensured that the mock is required lazily, variable names prefixed with `mock` are permitted.

This was fixed in a recent release. Please upgrade to Jest 17.

Thanks.

This is happening to me now that I have upgrade from 17 to 18.

Can you provide a repro?

Sorry I may have prematurely commented on this issue. It is because there was actual usage of variables outside the mock. Also the repo is private... I'll keep digging into it, but it looks like it was an expected error right now.

We allow the cov variable to pass this static analysis check here: https://github.com/facebook/jest/blob/new-docs/packages/babel-plugin-jest-hoist/src/index.js#L111
are you sure you are using babel-jest 18 and jest 18 together? Please let me know if you have a consistent repro in a repository on github and we'll fix this.

I am using<EMAIL_ADDRESS>and facing the same problem. You can check here:
https://travis-ci.org/chinesedfan/parrot-mocker/builds/368120227
But if removed this line, it will become OK.


UF-320: Preferences central UI + Admin Page

Merge together:
https://github.com/uberfire/uberfire/pull/530
https://github.com/droolsjbpm/kie-wb-common/pull/509
https://github.com/droolsjbpm/jbpm-form-modeler/pull/70
https://github.com/droolsjbpm/drools-wb/pull/318
https://github.com/droolsjbpm/jbpm-designer/pull/472
https://github.com/droolsjbpm/jbpm-console-ng/pull/506
https://github.com/droolsjbpm/optaplanner-wb/pull/112
https://github.com/droolsjbpm/livespark/pull/56
https://github.com/droolsjbpm/kie-wb-distributions/pull/378

+1


[SPARK-34630][PYTHON] Add typehint for pyspark.version

What changes were proposed in this pull request?
This PR adds the typehint of pyspark.version, which was mentioned in SPARK-34630.
Why are the changes needed?
There were some short discussion happened in https://github.com/apache/spark/pull/31823#discussion_r593830911 .
After further deep investigation on [1][2], we can see the pyspark.__version__ is added by setup.py, it makes __version__ embedded into pyspark module, that means the __init__.pyi is the right place to add the typehint for __version__.
So, this patch adds the type hint __version__ in pyspark/init.pyi.
[1] PEP-396 Module Version Numbers
[2] https://packaging.python.org/guides/single-sourcing-package-version/
Does this PR introduce any user-facing change?
No
How was this patch tested?

Disable the ignore_error on
https://github.com/apache/spark/blob/ee7bf7d9628384548d09ac20e30992dd705c20f4/python/mypy.ini#L132

Run mypy:

Before fix

(venv) ➜  spark git:(SPARK-34629) ✗ mypy --config-file python/mypy.ini python/pyspark | grep version
python/pyspark/pandas/spark/accessors.py:884: error: Module has no attribute "__version__"

After fix

(venv) ➜  spark git:(SPARK-34629) ✗ mypy --config-file python/mypy.ini python/pyspark | grep version

no output

cc @dannymeijer @zero323 @HyukjinKwon

Test build #137144 has started for PR 32110 at commit 2ec4f8d.

Test build #137144 has finished for PR 32110 at commit 2ec4f8d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137144/

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41723/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41723/

Merged to master and branch-3.1.
But @zero323 feel free to post-hoc review since you're the expert here :-)

I think it will require some changes. First of all __version__ is defined in python.version
https://github.com/apache/spark/blob/0e761c7307e780bd1d6cfe40e3d17ad003768411/python/pyspark/version.pyi#L19
so we shouldn't really annotate it directly in pyspark.__init__.
It is imported there, but not placed in __all__, so it formally not considered as exported.
We could cheat a bit and simply have
from pyspark.version import __version__  as __version__

but to be consistent, we should also add __version__ to __all__ and drop noqa: F401 from
https://github.com/apache/spark/blob/512f6258071666c3da07f6e6c6299d653a1cc370/python/pyspark/__init__.py#L63

@zero323 It makes sense to me, we should make the __version__ as exported, and I added a followup https://github.com/apache/spark/pull/32125 to fix it.


Test non-Western text in various forms and pages

(This used to be a bug issue which needed more information. I'm repurposing it into a unit-test issue.)
As CoralNet gets bigger, it becomes more important to handle non-ASCII and non-Western text (in general, Unicode) properly in any place the site supports text input. Even if the site's de facto language remains solely English, the data are coming in from institutions around the world. Any CharField, TextField, or uploaded .txt/.csv/other file is fair game.
By "handle", I mean do the following for each instance of text input:

If we decide to accept any Unicode character, make sure the characters are properly saved to the database and properly displayed on the site.
If we decide NOT to accept non-ASCII characters, or impose some other kind of restriction, then make sure we impose that restriction.

By "make sure", I mean write and run unit tests involving non-Western text input.
If we can be comprehensive about writing these kinds of tests, we'll also cover possibly one of the biggest tasks toward supporting Python 3 (#59 ).
Existing and to-do tests:

accounts

[x] User model - username: Non-ASCII not allowed, so usernames are simple and easy to read/type/differentiate for everyone.
[x] User model - email address: Non-ASCII not allowed, since such addresses are not common, and Django's built-in EmailValidator doesn't support non-ASCII yet (https://code.djangoproject.com/ticket/27029).

export

[x] Export annotations CSV: Non-ASCII image names and label codes OK.
[x] Export annotations CPC: Non-ASCII label codes OK.
[x] Export annotations CPC: Non-ASCII image names should be OK. This doesn't work yet because the local image filepath gets path-manipulated at some point, and in Python 2.x, pathlib2 doesn't support Unicode.
[x] Export covers: Non-ASCII image names and label codes OK.
[x] Export labelset: Non-ASCII label codes OK.

Known related bugs; these will be tracked in other issues, not here:

#125
#126
#202
OperationalError: (1267, "Illegal mix of collations (latin1_swedish_ci,IMPLICIT) and (utf8_general_ci,COERCIBLE) for operation '='") in the new-source page: This problem no longer exists. This happened in August 2016 with a location key 3 containing the Hawaiian punctuation mark ʻ info here. Relevant changes since then including moving from MySQL to PostgreSQL, and moving from "location keys" to aux. metadata fields.

Oh, the last remaining task (Export annotations CPC) was done last year in 06d374ab28a8493d031dba4942ac1f495cb2abb7.


Ensure subdomain is long enough

This further fixes the issue in #173 by appending the name from package.json to the subdomain if your username is < 4 characters.
cc @zeke

What if package.json doesn't have a name property?
What is the benefit of load-pkg over plain old require?
What if the app name + the package name is still under 4 chars? (Unlikely)

@zeke those are all good questions. I think trying to resolve this automagically is not worth it. I'm going to close this out.


Password reset/account recovery

Use MailChimp or the Network Canvas Gmail account for sending out reset emails?

Either should work. Path of least resistance. Google gmail accounts can be given app specific passwords for such things. I've done it in python with little fuss.

Got the email send working with Gmail, still working on a bug with the actual reset form.

Actually, the gmail account is not supposed to be used, generally. Can we do this with<EMAIL_ADDRESS>We can set up a legitimate mailbox with a new name (such as feedback-forum@networkcanvas.com), and provide smtp details. What would you need to make this work?

We can use any email address as long as there's a username/password for SMTP. So the @networkcanvas.com addresses should work fine.

Brilliant.

Just need to get the SMTP credentials for this to work.
Do you guys have a shared secret store like LastPass? Would be useful for some of the other deployment stuff too.

We actually don't. Can you message me on slack to try and get this set up? Afraid I am not too knowledgeable about it.

The feedback site is now sending password reset emails from<EMAIL_ADDRESS>


[Core] Implement repository pattern to save data offline

Cashing is required to save data to local storage.
Look at NYTimes Stoe project for pointer.

Use of store might be tricky and heavy, instead I'm thinking of leveraging okhttp interceptor to force caching.

Resources

https://krtkush.github.io/2016/06/02/caching-using-okhttp-part-2.html
https://gist.github.com/Tetr4/d10c5df0ad9218f967e0
https://medium.com/@I_Love_Coding/how-does-okhttp-cache-works-851d37dd29cd


[networks] Handle TCP_CLOSE/conntrack race

What does this PR do?
This PR addresses a race condition between TCP close events (read off the perf ring) and conntrack events, which causes network.ConnectionStats objects to occasionally miss IPTranslation objects in the context of short-lived connections.
The race can happen as follows:

a TCP connection is open and closed in quick succession (this can happen, for example, in a HTTP request);
The connection data is read off the perf ring;
Conntrack cache is looked up resulting in a miss;
Conntrack event is read off the netlink socket and added to the cache;

Aside from that I noticed a small issue in the connection merge codepath that, when combined to the race condition, can aggravate the problem. That may happen in the context of port re-use, and when multiple connections are merged, we can inadvertently override the IPTranslation field with a nil value. This was also fixed in this PR.
Motivation
Improve network data resolution.
Additional Notes
Anything else we should know when reviewing?
Describe how to test your changes
No QA required.
Checklist

[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] The need-change/operator and need-change/helm labels has been applied if applicable.
[ ] The appropriate team/.. label has been applied, if known.
[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The config template has been updated if applicable.

Note: Adding GitHub labels is only possible for contributors with write access.

Is there a way to write a regression test exposing the race condition and asserting the fix? Perhaps a conntracker with an artificial delay inserted?

Is there a way to write a regression test exposing the race condition and asserting the fix? Perhaps a conntracker with an artificial delay inserted?

@brycekahle Yes, that sounds reasonable. I'll give it a try


Packagist fallback for Composer

If the composer lookup fails, you could use the Packagist link instead: https://packagist.org/packages/{vendor}/{client}
eg. https://packagist.org/packages/google/apiclient

Would you like to submit a PR for that? https://github.com/github-linker/live-resolver/blob/master/src/resolver/composer.js

Implemented see https://github.com/github-linker/live-resolver/pull/4