[WIP] tooling: Add Github release manager

Commit Message:  tooling: Add Github release manager
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

coverage is here https://storage.googleapis.com/envoy-pr/17741/tooling/index.html

briefly reWIPping while i add paging for addittional release versions


Use downward API to pass current spec.nodeName to pod

The podInformerFactory uses filter key spec.nodeName to filter the pods that it should monitor. Up until now, this filter was set to the value of HOSTNAME. However, this is not reliable, as spec.nodeName can be overridden in kubernetes with --hostname-override and thus HOSTNAME and spec.nodeName do now necessarily always match. Instead, rely on a new custom environment variable NODENAME which is populated by the downward API.
Fixes https://issues.redhat.com/browse/OCPBUGS-10364

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.009%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
943

Relevant Lines:
1328

💛  - Coveralls

@andreaskaris for the unit tests, you'll have to change the signature of https://github.com/k8snetworkplumbingwg/whereabouts/blob/87e95f163a9284959daaaa42ad12335f5daa0de5/cmd/controlloop/controlloop.go#L109 allowing you to inject the client set into it.
K8S has a mock client set where you can provision it with the objects it would be able to retrieve: that way, you can write a negative and a positive test.

I addressed the unit tests a bit differently via the newDummyPodController @maiqueb @dougbtv mainly because I had already started rewriting it before I read the comments here. Let me know what you think - if that's not good enough, I can dig into @maiqueb 's suggestion


refac: Added metering point type and settlement method to request aggregated measure data

Description
Since wholesale are responsible for validating the metering point type and settlement method they need knowledge of them. Hence the mappings from metering point type and settlement method have been moved.
This pull request is dependent on: https://github.com/Energinet-DataHub/opengeh-wholesale/pull/1508
References

Codecov Report

Merging #555 (08feee3) into main (21978a1) will decrease coverage by 37.72%.
Report is 229 commits behind head on main.
The diff coverage is 32.74%.

@@             Coverage Diff             @@
##             main     #555       +/-   ##
===========================================
- Coverage   83.07%   45.36%   -37.72%
===========================================
  Files         456      288      -168
  Lines       10418     5593     -4825
===========================================
- Hits         8655     2537     -6118
- Misses       1438     2971     +1533
+ Partials      325       85      -240

Files
Coverage Δ

source/Application/Actors/CreateActorHandler.cs
0.00% <ø> (-85.72%)
:arrow_down:

...tion/Configuration/Authentication/Authenticated.cs
0.00% <ø> (-100.00%)
:arrow_down:

...n/Configuration/Authentication/NotAuthenticated.cs
0.00% <ø> (-100.00%)
:arrow_down:

...n/Configuration/Commands/CommandSchedulerFacade.cs
0.00% <ø> (-100.00%)
:arrow_down:

...Configuration/Commands/Commands/InternalCommand.cs
0.00% <ø> (-100.00%)
:arrow_down:

...e/Application/Configuration/Queries/QueryResult.cs
0.00% <ø> (ø)

...lication/Configuration/TimeEvents/ADayHasPassed.cs
0.00% <ø> (-100.00%)
:arrow_down:

...Configuration/TimeEvents/TenSecondsHasHasPassed.cs
0.00% <ø> (-75.00%)
:arrow_down:

...tgoingMessages/Common/OutgoingMessageExtensions.cs
0.00% <ø> (ø)

...ion/OutgoingMessages/Common/Xml/DocumentDetails.cs
100.00% <ø> (ø)

... and 46 more

... and 441 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fix(Range): rename locate to located

While finalizing the debugging interface, I noticed that the Range.locate is taking a location to create an element of located, not locating an element. I'd like to fix this API naming.

Range.locate becomes Range.located
Range.locate_opt becomes Range.located_opt
Range.locate_lex becomes Range.located_lex

I'm not sure if located_lex is the best name, but locate_lex is certainly wrong.

Are you saying that "giving a location to an element" shouldn't be described as "locating an element"?

@mikeshulman I thought "locate" means "finding the location of something when we don't know where it is", but in this case the location is explicitly given. I'm not insistent on this PR if native speakers feel otherwise.

In Merriam-Webster, the first definition of the transitive "locate" is "to determine or indicate the place, site, or limits of", but the second is "to set or establish in a particular spot".  So I think this use of "locate" is justifiable.

@mikeshulman Got it. Closing this now.


Added content_test.go to increase test coverage.

Proper pull request. For all the conversation please see pull request #1812.

As to commit message, see https://github.com/spf13/hugo/blob/master/CONTRIBUTING.md

Commit fixed. I didn't fix the commits as a whole. Now I got it. Thanks!

Yes, now it looks good (I modified your commit message slightly). Thanks for your contribution. Tests are very, very important.
Merged in e95f3af

Ahh thanks, I learned a lot from this. I'm working on the next batch already. I love writing tests actually.
Cheers @bep!


Unable to use the command line luis list querylogs

Tool
Name: LUIS
Version: 2.0.13
OS: Windows 10
Describe the bug
When using the command line "luis list querylogs" I have the following error:
Unknown resource: querylogs

LUIS Command Line Interface - © 2018 Microsoft Corporation

Gets the query logs of the past month for the application.

Usage:
luis list querylogs --appId <string>

Command arguments are:
 --appId <string>          The application ID.
 (required)

To Reproduce
Steps to reproduce the behavior:

Have a completed .luisrc file with your app infos
run the command luis list querylogs

Expected behavior
A list of the query logs of the past month
Additional context
I used Postman to test if I could fetch the logs with a simple Get and I was able to do it using my appId and subscription key. I also tried to add the appId to the command line and still no luck. I can use luis list versions ou luis get settings without any issue.
[bug]

Update: repo'd the issue, and hunted down the bug. Working on a fix

Update 2:
Part I
Short version: Looks like we need to find the right method from ms-rest-js module to call when handling response type "stream" in the LUIS CLI source code
As per Chris, I will connect with southworks team and see if they have any guidance

Part II
Detailing investigation here to update BF team and also not repeat myself when connecting with southworks via email to discuss further:
Cloning the LUIS CLI repo, then looking at the luis.js file, you can find the switch case where it handles the various "luis list" commands (e.g. luis list entities, luis list intents, etc.). There I think 17 or so luis list commands total.
Issue 1
There's no case for "querylogs", therefore triggering the familiar Unknown resource: querylogs error that @yanick1 reported
luis.js

Following all the other luis list commands cases, you can see that they call a list<CommandName>() method off the client (e.g. result = await client.model.listEntities(args.region, args.appId, args.versionId, args);
You can see when drilling down into the list<CommandName>() that everything the pattern of have 3 parts below:
list<Resource>(), HttpOperationResponse(), and OperationSpec
list<Resource>() method
  listEntities(azureRegion: Models.AzureRegions, appId: string, versionId: string): Promise<Models.EntityExtractor[]>;
  listEntities(azureRegion: Models.AzureRegions, appId: string, versionId: string, options: Models.ModelListEntitiesOptionalParams): Promise<Models.EntityExtractor[]>;
  listEntities(azureRegion: Models.AzureRegions, appId: string, versionId: string, callback: msRest.ServiceCallback<Models.EntityExtractor[]>): void;
  listEntities(azureRegion: Models.AzureRegions, appId: string, versionId: string, options: Models.ModelListEntitiesOptionalParams, callback: msRest.ServiceCallback<Models.EntityExtractor[]>): void;
  listEntities(azureRegion: Models.AzureRegions, appId: string, versionId: string, options?: Models.ModelListEntitiesOptionalParams, callback?: msRest.ServiceCallback<Models.EntityExtractor[]>): any {
    return msRest.responseToBody(this.listEntitiesWithHttpOperationResponse.bind(this), azureRegion, appId, versionId, options, callback);
  }

HttpOperationResponse method
  listEntitiesWithHttpOperationResponse(azureRegion: Models.AzureRegions, appId: string, versionId: string, options?: Models.ModelListEntitiesOptionalParams): Promise<msRest.HttpOperationResponse<Models.EntityExtractor[]>> {
    return this.client.sendOperationRequest(
      {
        azureRegion,
        appId,
        versionId,
        options
      },
      listEntitiesOperationSpec);
  }

OperationSpec
const listEntitiesOperationSpec: msRest.OperationSpec = {
  httpMethod: "GET",
  path: "luis/api/v2.0/apps/{appId}/versions/{versionId}/entities",
  urlParameters: [
    Parameters.azureRegion,
    Parameters.appId,
    Parameters.versionId0
  ],
  queryParameters: [
    Parameters.skip,
    Parameters.take
  ],
  responses: {
    200: {
      bodyMapper: {
        serializedName: "parsedResponse",
        type: {
          name: "Sequence",
          element: {
            type: {
              name: "Composite",
              className: "EntityExtractor"
            }
          }
        }
      }
    },
    default: {
      bodyMapper: Mappers.ErrorResponse
    }
  },
  serializer
};

Issue 2
There is no list<Resource>() (i.e. listQueryLogs) method in apps.ts. Only HttpOperationResponse and OperationSpec were built for querylogs in apps.ts
apps.ts
  downloadQueryLogsWithHttpOperationResponse(azureRegion: Models.AzureRegions, appId: string, options?: msRest.RequestOptionsBase): Promise<msRest.HttpOperationResponse<void>> {
    return this.client.sendOperationRequest(
      {
        azureRegion,
        appId,
        options
      },
      downloadQueryLogsOperationSpec);
  }

const downloadQueryLogsOperationSpec: msRest.OperationSpec = {
  httpMethod: "GET",
  path: "luis/api/v2.0/apps/{appId}/querylogs",
  urlParameters: [
    Parameters.azureRegion,
    Parameters.appId
  ],
  responses: {
    200: {
      bodyMapper: {
        serializedName: "parsedResponse",
        type: {
          name: "Stream"
        }
      }
    },
    default: {}
  },
  serializer
};

Issue 3
You cannot use msRest.responseToBody() method in making request to list query logs, because it is a "Stream" type.
It actually appears to the ONLY stream type in the LUIS CLI, as far as I could find looking through all the OperationSpecs, probably because it's trying to download the history of all the endpoint hits to an app for the past 30 days.
In trying to mimic the requests made for all the other luis list <resource> commands, I did try implementing a case for "querylogs" and creating a method off the client for listQueryLogs() doing the following
luis.js
                case "querylogs":
                    result = await client.apps.listQueryLogs(args.region, args.appId, args);
                    break;

apps.ts
    listQueryLogs(azureRegion: Models.AzureRegions, appId: string,options?: msRest.RequestOptionsBase, callback?: msRest.ServiceCallback<void>): any {
      return msRest.responseToBody(this.downloadQueryLogsWithHttpOperationResponse.bind(this), azureRegion, appId, options, callback);
    }

But this merely returns "OK" in the console, not a list of the query logs.
this is because downloadQueryLogsWithHttpOperationResponse()  results have undefined asBodyText and parsedBody properties
Looking in the MS Rest module utils file, you can see that responseToBody() is not the right method to handle Stream type then due to lack of parsedBody

Instead of asBodyText and parsedBody being defined, downloadQueryLogsWithHttpOperationResponse has results that define readableStreamBody property that has lots of detail in object value

Having never worked with ms-rest-js, I'm unfamiliar with what method would be the right to use in place of responseToBody to send the operation request (sendOperationRequest). This is what I'm currently investigating.

Part III
Reconfirming what @yanick1 had mentioned earlier, the LUIS Programmatic API portion works just fine
Tested via Postman, browser, and using creating a small node project to make a call directly to LUIS API without MS Rest. CSV downloaded successfully
Issue is definitely in the LUIS CLI, specifically with using ms rest properly


Add Costume type to model

Color names based on ssbwiki.com, but I made sure Red, Green, and Blue existed for every character. I plan to add colloquial names if #8 gets merged.
I removed derive implementations of Default/Deserialize for the Player type because my change caused conflicts. It doesn't seem like those traits made much sense in the first place.
It might make sense to move the default, red, green, blue functions to the External type, but I left it how it is for now.

Looks great. Thanks!
How do you anticipate red() & co being used? I assume for teams, but to do what exactly?

I honestly didn't have a solid use case in mind. I was having fun with macros and it seemed like a simple addition. I understand wanting to keep the api streamlined, so I'm not sentimental about those functions.
Maybe a niche use case would be searching for games with red characters but you don't care which character in particular.

I honestly didn't have a solid use case in mind. I was having fun with macros and it seemed like a simple addition. I understand wanting to keep the api streamlined, so I'm not sentimental about those functions.

I'd rather leave them out for now, in that case. We can always add them if there's demand, but removing them would break backwards compatibility.

Ok I removed them for now. I plan on using this costume feature. If I find that the red/green/blue functions would be useful for me I will bug you to add them back in at that point.

Rebased and squashed

One last thing: could you replace spaces with tabs? Sorry I haven't gotten around to configuring rustfmt yet.

I'm not seeing any space indents on this branch

One last thing: could you replace spaces with tabs? Sorry I haven't gotten around to configuring rustfmt yet.

I'm not seeing any space indents on this branch

Oh my bad, Firefox was misleading me. Sorry for the noise.


Unable to update via apt : mirror sync in progress ?

Since a few days i can't update vscode via apt update

OS Version: Ubuntu 20.04

E: Failed to fetch https://packages.microsoft.com/repos/vscode/dists/stable/main/binary-amd64/Packages.bz2  File has unexpected size (201047 != 200914). Mirror sync in progress? [IP: <IP_ADDRESS> 443]    Hashes of expected file:     - Filesize:200914 [weak]     - SHA512:a91d388504b56c875620f73b618813be1c127d6bfe96406cc6e9a39fdf33f1dd2c8ecbda4a636f0c86bc18005e4f5d0683da0c02ba13c555cef8d11eb83412fe     - SHA256:1d39334cb2779572dd17869a05d3ed3f1d57992cc9755a34a4d2622132b19f57     - SHA1:d5224a98d6adfb4ab65d66aa803f9688a6503080 [weak]     - MD5Sum:2fc601c3aa23f067ef305bfb7f7086e7 [weak]    Release file created at: Mon, 28 Sep 2020 06:53:06 +0000 E: Some index files failed to download. They have been ignored, or old ones used instead.

Same here, ubuntu version as @atmaniak, region Tenerife/Spain.

Same issue here since this morning on 2 debian systems :  Debian GNU/Linux 9.13 (stretch), and the second one is a Debian buster
Réception de:8 http://packages.microsoft.com/repos/vscode stable/main amd64 Packages [201 kB]
Err:8 http://packages.microsoft.com/repos/vscode stable/main amd64 Packages
Writing more data than expected (201047 > 200914)
Hashes of expected file:

Filesize:200914 [weak]
SHA512:a91d388504b56c875620f73b618813be1c127d6bfe96406cc6e9a39fdf33f1dd2c8ecbda4a636f0c86bc18005e4f5d0683da0c02ba13c555cef8d11eb83412fe
SHA256:1d39334cb2779572dd17869a05d3ed3f1d57992cc9755a34a4d2622132b19f57
SHA1:d5224a98d6adfb4ab65d66aa803f9688a6503080 [weak]
MD5Sum:2fc601c3aa23f067ef305bfb7f7086e7 [weak]
Release file created at: Mon, 28 Sep 2020 06:53:06 +0000
142 ko réceptionnés en 0s (232 ko/s)
Lecture des listes de paquets... Fait
E: Impossible de récupérer http://packages.microsoft.com/repos/vscode/dists/stable/main/binary-amd64/Packages.bz2  Writing more data than expected (201047 > 200914)
Hashes of expected file:
- Filesize:200914 [weak]
- SHA512:a91d388504b56c875620f73b618813be1c127d6bfe96406cc6e9a39fdf33f1dd2c8ecbda4a636f0c86bc18005e4f5d0683da0c02ba13c555cef8d11eb83412fe
- SHA256:1d39334cb2779572dd17869a05d3ed3f1d57992cc9755a34a4d2622132b19f57
- SHA1:d5224a98d6adfb4ab65d66aa803f9688a6503080 [weak]
- MD5Sum:2fc601c3aa23f067ef305bfb7f7086e7 [weak]
Release file created at: Mon, 28 Sep 2020 06:53:06 +0000
E: Le téléchargement de quelques fichiers d'index a échoué, ils ont été ignorés, ou les anciens ont été utilisés à la place.

Same ubuntu version 20.04, region: India

same issue for Ubuntu 20.04. region: France

Same on Ubuntu 20.04.1, region: Czech

Same issue here. Elementary OS 5.1.7 (hera), region: Denmark

Duplicate of: https://github.com/microsoft/vscode/issues/107773

The same issue. Debian Bullseye.

same issue on eos 5.1.7 hera

Same issue here. KUbuntu 20.04.1. Region: Serbia.
E: Failed to fetch http://packages.microsoft.com/repos/vscode/dists/stable/main/binary-amd64/Packages.bz2  File has unexpected size (201420 != 200914). Mirror sync in progress?

Well, it looks like it's fixed now. :)

Yes for sure...
Same problem:
Zorin OS 15.3, India

This works for me (connecting from Switzerland).
@KushalSM Could you retry? This might take some time to reach all regions.

Checked with the repo team. This was fixed almost 5 hours ago. Anyone still seeing this, please let us know.

same issue on eos 5.1.7 hera
this fixed on this morning

Restored (debian buster). Thanks !

Back to normal !

@chrmarti
Hey its all fine now. Thx.
Cheers!!!

Hello.
The same issue now.
cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.1 LTS"

Error:
Get:15 http://packages.microsoft.com/repos/vscode stable/main amd64 Packages [204 kB]
Err:15 http://packages.microsoft.com/repos/vscode stable/main amd64 Packages
  File has unexpected size (203820 != 203585). Mirror sync in progress? [IP: <IP_ADDRESS> 80]
  Hashes of expected file:
   - Filesize:203585 [weak]
   - SHA512:070b1469cbfcc74fbe83b5b8bd1af9707f9f0ad520d4bf161340532c8bec4fd31812cf9242a81fe54e805837495281bcece1edb925104ff817191e068970863c
   - SHA256:7e6635480cc4a9463de3167215ad504c8721bd43d9c40b4ad71b44f60380757d
   - SHA1:9baf9b98b78b41be5c361ba7825f016e1abd31d2 [weak]
   - MD5Sum:f509c48fcb219ac06f64fbec78be0dd5 [weak]
  Release file created at: Thu, 15 Oct 2020 18:21:44 +0000
Fetched 10,6 kB in 1s (8 524 B/s)
Reading package lists... Done
E: Failed to fetch http://packages.microsoft.com/repos/vscode/dists/stable/main/binary-amd64/Packages.bz2  File has unexpected size (203820 != 203585). Mirror sync in progress? [IP: <IP_ADDRESS> 80]
   Hashes of expected file:
    - Filesize:203585 [weak]
    - SHA512:070b1469cbfcc74fbe83b5b8bd1af9707f9f0ad520d4bf161340532c8bec4fd31812cf9242a81fe54e805837495281bcece1edb925104ff817191e068970863c
    - SHA256:7e6635480cc4a9463de3167215ad504c8721bd43d9c40b4ad71b44f60380757d
    - SHA1:9baf9b98b78b41be5c361ba7825f016e1abd31d2 [weak]
    - MD5Sum:f509c48fcb219ac06f64fbec78be0dd5 [weak]
   Release file created at: Thu, 15 Oct 2020 18:21:44 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.

Same issue, Ubuntu 18.04.5 LTS:
E: Failed to fetch http://packages.microsoft.com/repos/vscode/dists/stable/main/binary-amd64/Packages.bz2  File has unexpected size (203820 != 203585). Mirror sync in progress? [IP: <IP_ADDRESS> 80]
   Hashes of expected file:
    - Filesize:203585 [weak]
    - SHA512:070b1469cbfcc74fbe83b5b8bd1af9707f9f0ad520d4bf161340532c8bec4fd31812cf9242a81fe54e805837495281bcece1edb925104ff817191e068970863c
    - SHA256:7e6635480cc4a9463de3167215ad504c8721bd43d9c40b4ad71b44f60380757d
    - SHA1:9baf9b98b78b41be5c361ba7825f016e1abd31d2 [weak]
    - MD5Sum:f509c48fcb219ac06f64fbec78be0dd5 [weak]
   Release file created at: Thu, 15 Oct 2020 18:21:44 +0000

Still having problems. Location: India
`Err:6 https://packages.microsoft.com/repos/vscode stable/main amd64 Packages
File has unexpected size (203820 != 203585). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:

Filesize:203585 [weak]
SHA512:070b1469cbfcc74fbe83b5b8bd1af9707f9f0ad520d4bf161340532c8bec4fd31812cf9242a81fe54e805837495281bcece1edb925104ff817191e068970863c
SHA256:7e6635480cc4a9463de3167215ad504c8721bd43d9c40b4ad71b44f60380757d
SHA1:9baf9b98b78b41be5c361ba7825f016e1abd31d2 [weak]
MD5Sum:f509c48fcb219ac06f64fbec78be0dd5 [weak]
Release file created at: Thu, 15 Oct 2020 18:21:44 +0000
Fetched 3,959 B in 1s (2,716 B/s)
Reading package lists... Done
E: Failed to fetch https://packages.microsoft.com/repos/vscode/dists/stable/main/binary-amd64/Packages.bz2  File has unexpected size (203820 != 203585). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:
- Filesize:203585 [weak]
- SHA512:070b1469cbfcc74fbe83b5b8bd1af9707f9f0ad520d4bf161340532c8bec4fd31812cf9242a81fe54e805837495281bcece1edb925104ff817191e068970863c
- SHA256:7e6635480cc4a9463de3167215ad504c8721bd43d9c40b4ad71b44f60380757d
- SHA1:9baf9b98b78b41be5c361ba7825f016e1abd31d2 [weak]
- MD5Sum:f509c48fcb219ac06f64fbec78be0dd5 [weak]
Release file created at: Thu, 15 Oct 2020 18:21:44 +0000`

I have the same issue in USA
Err:38 https://packages.microsoft.com/ubuntu/18.04/prod bionic/main amd64 Packages
File has unexpected size (141364 != 141252). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:

Filesize:141252 [weak]
SHA512:dffbaf15f537ae643df110b279248993b2828e99352dff51a5df9648cf39531ffb7022ec2f0d9e553287c9889addcb5e734d9df95f749c456db04fd328a67ef4
SHA256:6757f7f09fb2a0491657aac899faa29bd67fe7b9dfd849844f7fafca9afcc33e
SHA1:37be76673db70e808db91294af54fddf476aae7f [weak]
MD5Sum:ecc0d5d8d9373eee7aeceda58d7be8f9 [weak]
Release file created at: Tue, 13 Oct 2020 22:35:00 +0000
Get:39 https://cli-assets.heroku.com/apt ./
E: Failed to fetch https://packages.microsoft.com/ubuntu/18.04/prod/dists/bionic/main/binary-amd64/Packages.bz2  File has unexpected size (141364 != 141252). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
Hashes of expected file:
- Filesize:141252 [weak]
- SHA512:dffbaf15f537ae643df110b279248993b2828e99352dff51a5df9648cf39531ffb7022ec2f0d9e553287c9889addcb5e734d9df95f749c456db04fd328a67ef4
- SHA256:6757f7f09fb2a0491657aac899faa29bd67fe7b9dfd849844f7fafca9afcc33e
- SHA1:37be76673db70e808db91294af54fddf476aae7f [weak]
- MD5Sum:ecc0d5d8d9373eee7aeceda58d7be8f9 [weak]
Release file created at: Tue, 13 Oct 2020 22:35:00 +0000
E: Some index files failed to download. They have been ignored, or old ones used instead.
Error: The process 'sudo' failed with exit code 100

Just got the same issue:( Siberia, Russia.

Works now

This was fixed around Friday 6:15am UTC. Let us know if you still see any issues. Thanks.

This was fixed around Friday 6:15am UTC. Let us know if you still see any issues. Thanks.

It's fixed now. Thanks 👍

same issue on ubuntu:20.04

This worked for me on Ubuntu 20.04. First, remove all the vscode related files in source.list.d and then update your packages.
sudo rm -rf /etc/apt/sources.list.d/vscode.list*
sudo apt-get update

I spent more than 6 hours searching for a solution and finally above mentioned solution worked.
Hope this helps.

Same issue, Ubuntu 20.04 Argentina!


Fixed the lack of default AttributeDict argument in to_dot

Lack of default attributes causes errors on calls without explicit AttributeDict passed, thus making plot() for graphs unusable

Thanks for the fix. Could you add a test for this while you're at it?

Tests would be good, but this is a recent regression, so I'm merging anyway.


Settings list page

[x] Make the settings page match the one we have for Cluster Manager (remove Age, add Value field to table and show the value for that setting)  We have a component you should be using for this... connect with Vincent.

Table and list page should look like this:

The one in Rancher is just a generic view from the custom resource definition.  You can edit your CRD to include value as one of the printer columns so that it shows up in kubectl get ... too.

Verified fixed


Add support for WitsmlTubular

Request Template Witsml Explorer
Thank you for contributing to WITSML Explorer!
Before submitting this PR, please fill in the following template.
Fixes
This pull request fixes # (issue)
Description
Adding WitsmlTubular support.
Type of change
List the types of change. Remove from the list any points which are not necessary

Bugfix
New feature (non-breaking change which adds functionality)
Enhancement of existing functionality
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

Impacted Areas in Application
List general components of the application that this PR will affect:

Checklist:
Please tick all the boxes or remove the ones that aren't needed and explain why
Communication

[ ] PR is related to an issue
[ ] I have made corresponding changes to the documentation

Code quality

[x] Code follows the style guidelines
[x] I have self-reviewed my code
[x] No new warnings are generated

Test coverage

[x] Existing tests pass
[ ] New code is covered by passing tests

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Should the update of the console application be a separate PR?

@stigrus - Please fill in the first sections of the PR template above.


A component is contentEditable and contains children managed by React warning

When integrating draft-js into an existing project I get

Warning: A component is contentEditable and contains children managed by React. It is now your responsibility to guarantee that none of those nodes are unexpectedly modified or duplicated. This is probably not intentional.

I am using the rich text example, and that gives me this warnings:

Warning: Each child in an array or iterator should have a unique "key" prop. Check the render method of BlockStyleControls. See https://fb.me/react-warning-keys for more information.
Warning: Each child in an array or iterator should have a unique "key" prop. Check the render method of InlineStyleControls.

using the rich.htmlexample from the repository

Known problem. http://facebook.github.io/draft-js/docs/advanced-topics-issues-and-pitfalls.html#react-contenteditable-warning
React 15 will give a way to silence this.

The key warnings could be fixed though.

Hello, I am still having this warning, but I checked the draft-js code (0.7.0 version) and it has the  suppressContentEditableWarning: true, prop in the file.
I also have react 15.3.1 so, I am wondering what could be wrong.

@jmaguirrei You must not actually have react-dom 15.3.1. Check npm ls for the full dependency tree.

@spicyj Thanks. I wasn't with react-dom 15.3.1, but after upgrade I am still having same warning.
If I go to the function the console shows in ReactDOMComponent.js this is:
function assertValidProps(component, props) {
  if (!props) {
    return;
  }
  // Note the use of `==` which checks for null or undefined.
  if (voidElementTags[component._tag]) {
    !(props.children == null && props.dangerouslySetInnerHTML == null) ? process.env.NODE_ENV !== 'production' ? invariant(false, '%s is a void element tag and must neither have `children` nor use `dangerouslySetInnerHTML`.%s', component._tag, component._currentElement._owner ? ' Check the render method of ' + component._currentElement._owner.getName() + '.' : '') : _prodInvariant('137', component._tag, component._currentElement._owner ? ' Check the render method of ' + component._currentElement._owner.getName() + '.' : '') : void 0;
  }
  if (props.dangerouslySetInnerHTML != null) {
    !(props.children == null) ? process.env.NODE_ENV !== 'production' ? invariant(false, 'Can only set one of `children` or `props.dangerouslySetInnerHTML`.') : _prodInvariant('60') : void 0;
    !(typeof props.dangerouslySetInnerHTML === 'object' && HTML in props.dangerouslySetInnerHTML) ? process.env.NODE_ENV !== 'production' ? invariant(false, '`props.dangerouslySetInnerHTML` must be in the form `{__html: ...}`. Please visit https://fb.me/react-invariant-dangerously-set-inner-html for more information.') : _prodInvariant('61') : void 0;
  }
  if (process.env.NODE_ENV !== 'production') {
    process.env.NODE_ENV !== 'production' ? warning(props.innerHTML == null, 'Directly setting property `innerHTML` is not permitted. ' + 'For more information, lookup documentation on `dangerouslySetInnerHTML`.') : void 0;
    process.env.NODE_ENV !== 'production' ? warning(props.suppressContentEditableWarning || !props.contentEditable || props.children == null, 'A component is `contentEditable` and contains `children` managed by ' + 'React. It is now your responsibility to guarantee that none of ' + 'those nodes are unexpectedly modified or duplicated. This is ' + 'probably not intentional.') : void 0;
    process.env.NODE_ENV !== 'production' ? warning(props.onFocusIn == null && props.onFocusOut == null, 'React uses onFocus and onBlur instead of onFocusIn and onFocusOut. ' + 'All React events are normalized to bubble, so onFocusIn and onFocusOut ' + 'are not needed/supported by React.') : void 0;
  }
  !(props.style == null || typeof props.style === 'object') ? process.env.NODE_ENV !== 'production' ? invariant(false, 'The `style` prop expects a mapping from style properties to values, not a string. For example, style={{marginRight: spacing + \'em\'}} when using JSX.%s', getDeclarationErrorAddendum(component)) : _prodInvariant('62', getDeclarationErrorAddendum(component)) : void 0;
}

// and below in the file...

var RESERVED_PROPS = {
  children: null,
  dangerouslySetInnerHTML: null,
  suppressContentEditableWarning: null
};

// and in npm ls
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>
Any idea why this is happening?

No, I'm not sure.

@jmaguirrei did you succeed? I'm having the same issue

Yes, Just use suppressContentEditableWarning in your component.
Be aware that this is not related with draftjs but with the use of any div that is content editable, so you may ne having this warning at another component.
That was my case...
Let me know, glad to help.
BTW I dont know how to make a mention on mobile phone... :( do you?

@jmaguirrei like this? It's not working
          editorState={ this.state.editorState }
          onChange={ this.onTextChange }
          suppressContentEditableWarning={ true }
        />```

@martriay Try putting the property at the containing div of the editor, like:
  render() {

    const
      { isSelectionActive, selectionCoords } = this,
      { size, owner } = this.props,
      readOnly = !owner;

    return (

          <Editor
            editorState={this.state.editorState}
            onChange={this.onChange}
            handleKeyCommand={this.handleKeyCommand}
            keyBindingFn={this.myKeyBindingFn}
            handleBeforeInput={this.handleBeforeInput}
            readOnly={readOnly}
            placeholder={this.placeholder}
            ref='editor'
          />

        {
          isSelectionActive &&

            MySelection

        }

    );
  }

@jmaguirrei thank you very much! But I ended suppressing it manually:
console.error = (function(_error) {
  return function(message) {
    if (typeof message !== 'string' || message.indexOf('component is `contentEditable`') === -1) {
      _error.apply(console, arguments)
    }
  }
})(console.error)

@martriay Nice solution to have as a general escape... Thanks also!

@jmaguirrei suppressContentEditableWarning is work fine for me, Thanks!

Encountered this issue on Quilljs, can you guys help me to remove the error message?


add new property TuneFastDeviceClass

Signed-off-by: crombus<EMAIL_ADDRESS>

@jarrpa I ran make gen-latest-csv but crd changes were not done. What can I do to make those changes

I think we should make these changes in two commits first one for the move part and second one for add part.

@jarrpa the ci test is failing at govet. need help there.

@crombus since the implementation of throttleStorageDevices method is modified, it's necessary to make necessary changes in TestThrottleStorageDevices (throttleStorageDevices is invoked here ) under storagecluster_controller_test.go.

Test failures are valid. Please rebase and resolve. Always run make ocs-operator-ci before any push.

the PR is dependent on https://github.com/openshift/ocs-operator/pull/912. rook 1.5.1. I'll rebase it.
The failures are because of govet.

the PR is dependent on #912. rook 1.5.1. I'll rebase it.
The failures are because of govet.

#912 has merged now @crombus - you can rebase

Not strictly required for a 4.7 feature, afaict, but would be good to have before feature freeze...

/test ocs-operator-ci

/test ocs-operator-bundle-e2e-aws

/test ocs-operator-bundle-e2e-aws

/retest

/test ocs-operator-ci

/cherry-pick release-4.6

/cherry-pick release-4.6

@crombus https://bugzilla.redhat.com/show_bug.cgi?id=1909793 still doesn't have all the acks. Pls do not merge it into 4.6 yet.
Also, you should first backport it to 4.7

@crombus https://bugzilla.redhat.com/show_bug.cgi?id=1909793 still doesn't have all the acks. Pls do not merge it into 4.6 yet.
Also, you should first backport it to 4.7

@agarwal-mudit this feature is already in 4.7.
And this will require manual BP of the two PR's mentioned in BZ.
I would say it is tricky to BP.

@agarwal-mudit this feature is already in 4.7.
And this will require manual BP of the two PR's mentioned in BZ.
I would say it is tricky to BP.

@agarwal-mudit this feature is already in 4.7.
And this will require manual BP of the two PR's mentioned in BZ.
I would say it is tricky to BP.

Ok, then lets not backport it to 4.6
Probably we can GA Azure without this feature. @obnoxxx

@agarwal-mudit this feature is already in 4.7.
And this will require manual BP of the two PR's mentioned in BZ.
I would say it is tricky to BP.

Ok, then lets not backport it to 4.6
Probably we can GA Azure without this feature. @obnoxxx


Update font-xkcd to latest

The location of the font moved into a subfolder (./build/ ➡ ./xkcd/build/).

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

Thanks for the PR!


Add Python 3.10 support (close #254)

This PR addresses issue #254 and adds support for Python 3.10 by dropping the pycontracts library (not updated for some time and incompatible with 3.10 at this time) and adding some other features that require support for Python 2 to be dropped.
The full list of changes is:

Remove pycontracts library that wasn't compatible with Python 3.10
Add Python type hints to all function arguments and return values
Remove support for Python 2 due to the type hints
Add custom contracts (e.g., non_empty_string) evaluated at the start of functions
Run through Flake8 linter and correct style
Add Dockerfile for running tests under all supported Python versions

Note:

Python type hints are not enforced during runtime (they can be checked using mypy static type checking or through IDEs such as PyCharm) so some types of issues that previously raised errors no longer do so

in particular, this is the case for instance type matching, I didn't implement contracts for checking instance types – I don't think this is a standard behaviour in Python and IDEs should help with these types of errors using the type hints
I tried to add custom contracts for non-instance-type checks such non-empty strings, values larger than 0, and non-empty lists

There is a potentially breaking change because we no longer raise ContractNotRespected error (from pycontracts) when contracts are violated but the standard ValueError

Coverage decreased (-1.4%) to 96.072% when pulling abd9ef14916326d7222dbc2e68f4e6e548733b12 on issue/254-python_310 into 8fc7f1e5ed9c0e1b1faf8b477be8412b4e2630c2 on master.

I'm also assigning @adatzer to review this. I'll hopefully get to it too shortly but I'd love her input on this too given she has been the most recent maintainer on this tracker.

Thanks a lot @adatzer for the thorough review! Those were really useful comments and also uncovered a few important problems.
Not sure about removing the :type pycontracts comments – they still have some value in documenting the types although now the type documentation is duplicated in the type hints too... Do you know if they are used when generating any API docs?


Implement Building Blocks for Security S2

This includes:

[x] raw x25519 key handling
[x] AES-CMAC function
[x] AES-128 CTR_DRBG Pseudo-Random Number Generator
[ ] SPAN Nonce generator
[ ] MPAN Nonce generator

fixes: #1122

Wow @AlCalzone Seems you had some busy days :open_mouth:


[WIP] Frontend monorepo

CATEGORY

[x] Build / Development Environment

SUMMARY
@kristw @rusackas This is what I meant in #9187. Frontend packages will be organized as such:
├── lerna.json
├── package-lock.json
├── package.json
├── packages
│   ├── superset-app
│   ├── superset-ui-demo
│   └── superset-ui-plugins-demo
├── superset-ui
│   ├── chart
│   ├── chart-composition
│   ├── color
│   ├── ...
└── superset-ui-plugins
    ├── egacy-rose
    ├── icicle-event
    ├── legacy-calendar
    ├── legacy-chord
    ├── ...

There's enough isolation for optional plugins, but development imports and plugin update workflows can be made very easy... There's obviously more work needed to actually make this work, but I think it's a direction worth pursuing. What do you think?

On the one hand, it might make life easier, as I don't love the double-PR flow needed to make changes right now, and this resolves at least that issue. On the other hand, there are two potential issues that jump to the top of my mind: 1) this is a step away from having an ecosystem of plugins from 3rd party repos, and 2) I think that one of the reasons these were moved to their own repo in the first place had to do with isolating the licenses of certain packages from the core Apache release. I worry that this'll be another difficult thing to unwind later, but concede that it may make life easier for today. I'm curious what @kristw and @mistercrunch think of the idea.

@rusackas

this is a step away from having an ecosystem of plugins from 3rd party repos

Right. A 3rd party repo will run into the same problem and we need to build dev tools to support anyway. Better dogfood that.

I think that one of the reasons these were moved to their own repo in the first place had to do with isolating the licenses of certain packages from the core Apache release. I worry that this'll be another difficult thing to unwind later, but concede that it may make life easier for today.

+1
Another reason of moving out is we are using most (if not all) of the quota for running multiple ci jobs in parallel per repo. We spent lots of effort in cutting down ci time at one point to improve the developer experience in this repo. If we bring all these packages into it, it will bring a few more ci jobs along.

@ktmud Now that your (amazing) npm link work is in, I guess we can close this one? Up to you :D

I'm still not liking the double PR workflow and the manual process of going back and forth editing package versions while maintaining two open PRs.  But I guess that's a small price to pay to get the benefits that motivated the split in the first place.

I'm still not liking the double PR workflow and the manual process of going back and forth editing package versions while maintaining two open PRs. But I guess that's a small price to pay to get the benefits that motivated the split in the first place.

Agreed... hopefully we can get ALL of the plugins' parts into their own repo, and remove/reduce double-PR pains.


Initial Course Model

WHAT DOES THIS PR DO?

Creates the initial course model within the School app.
Writes the simple test case for this model in test_models.py

WHAT ISSUES ARE RELATED TO THIS PR?

#86

HOW DO I TEST OUT THIS PR?

There is no functionality of this model yet, only a test within test_models.py exists.

ADDITIONAL COMMENTS
This PR creates questions we must answer for the backend.

Do we desire any more fields for a course model?
Are there any changes we desire in the current field design?
Not so much a question, but professor should be tied to a course, once professors are implemented into the API database.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (c453f92) 100.00% compared to head (f5642ba) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #97   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           21        21
  Lines          313       322    +9
=========================================
+ Hits           313       322    +9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


expire unused ids optional

If a bridge is published without any accessories and the accessories are added later the functionality to expire unused ids in the IdentifierCache is removing all the ID's and that's creating mess on the iOS later when the accessories are dynamically added. So i added functionality to disable the 'expire unusdes ids' also a check in the publish weather the accessory is bridge and the length of bridgedAccessories in the bridge is 0 then disable the 'expire unusdes ids'.

Thanks :)


added release notes, modified kylin and coprocessor upgrade guides

Can one of the admins verify this patch?

本 PR 先关闭。

I agree with @shaofengshi's comments, and in that way, this PR is not that necessary, thus I closed this PR.


Wrong URL generated for newly created buckets

I've experienced this bug in the past, and can recreate it right now:
Uploading a file (I do it with jquery.fileupload but that doesn't matter) will fail because the target URL has no region in it. AWS will respond with 307.
Screenshot

Here you can see an OPTIONS request to https://estiu-uploads-development.s3.amazonaws.com/, while it should have been performed to https://estiu-uploads-development.s3-us-west-2.amazonaws.com/.
pry session
[1] pry(#<EventsController>)> @uploader.direct_fog_url
=> "https://estiu-uploads-development.s3.amazonaws.com/"

Workaround
Wait 1 day or so until AWS does some DNS magic I think

We've managed to overcome this issue by temporary hacking uploader method:
def direct_fog_url
    "https://bucker-name-here.s3-us-west-2.amazonaws.com/"
end


sign_in method not works on a post request

I'm using rails 4 and devise 3.5.2, the login code looks like below:
  def token_login
    auth_token = AuthToken.find_by(token: params[:auth_token])
    if auth_token
      @user = auth_token.user
      sign_in(:user, @user)
      render template: '/api/user'
    else
      return render_error API_ERROR_INVALID_KEY, "invalid token"
    end
  end

It works well if I call this method with get request, but sign_in would have no effect (without any error thrown) if I call this method with AJAX post request, is devise designed to work like this or I'm missing something?

You need to give us more information on how to reproduce this issue, otherwise there is nothing we can do. Please read CONTRIBUTING.md file for more information about creating bug reports. Thanks!

The reason is from protect_from_forgery with: :null_session from rails, it will remove any cookie from the response.


Allow explicit configured plugins to execute on master as well

Extended the MasterPromoteExtension to not only retain the hardcoded gitflow-helper-maven-plugin itself and the maven-deploy-plugin, or the plugins related to explicit supplied command-line goals, but also allow a project to configure additional plugins that may also execute on master/support.
This serves two cases:

It solves the issue #127 (though admittedly it still would be more clean to be able to download the pom file from the release repo)
It can serve as a basic extension point to allow other "things" to happen on a master build, that are beyond the scope of deploying maven artifacts (ie have some antrun script run)

In the following example the flatten-maven-plugin has two executions: flatten-in-master and flatten-other
Only the flatten-in-master.
Will ot work with this PR?
<plugin>
	<groupId>org.codehaus.mojo</groupId>
	<artifactId>flatten-maven-plugin</artifactId>
	<executions>
		<execution>
			<id>flatten-in-master</id>
			<goals>
				<goal>flatten</goal>
			</goals>
			<phase>process-resources</phase>
		</execution>
			<id>flatten-other</id>
			<goals>
				<goal>flatten</goal>
			</goals>
			<phase>process-resources</phase>
		</execution>
	</executions>
</plugin>
<plugin>
	<plugin>
		<groupId>com.e-gineering</groupId>
		<artifactId>gitflow-helper-maven-plugin</artifactId>
		<configuration>
			<retainPlugins>
				<retainPlugin>org.codehaus.mojo:flatten-maven-plugin:flatten-in-master</retainPlugin>
			</retainPlugins>
		</configuration>
	</plugin>
</plugin>

Thank you @glimmerveen for this PR.
In the following example the flatten-maven-plugin has two executions: flatten-in-master and flatten-other Only the flatten-in-master should be executed in the master build.
Will this use case work with this PR?

In its current form no; it currently retains all executions of a plugin. I added this capability on top of existing logic that ensured that plugins activated through command line arguments would be retained. That existing logic centered around the idea that no plugins should be allowed to execute on master, and achieved that by removing complete Plugin 'activations'. This PR added a third way to retain plugins (on top of the default excludes, and command line activated plugins).
More fine-grained control is definitely possible, but being able to retain specific execution(s) may require a bit of refactoring of this logic.

Thank you. I created issue #136 for this feature.

Thank you. I created issue #136 for this feature.

Based on your question I took the liberty to update the logic to support your use-case as well. See PR #137


Minor: Remove import.meta.env workaround, use Markdown syntax

What kind of changes does this PR include?

[X] Minor content fixes (broken links, typos, etc.)
[ ] New or updated content
[ ] Translated content
[X] Changes to the docs site code
[ ] Something else!

Description

Updates Astro to 1.0.0-beta.51, which includes a fix that allows using import.meta.env directly in Markdown code. As this fix made our clever workaround of using an ImportMetaEnv Astro component unnecessary, I've removed the component and replaced all references to it with plain Markdown.
I've also converted unnecessary HTML tags (<p>, <ul>/<li> etc.) to their proper Markdown variants.
Marking this as "Minor" to avoid affecting translation up-to-dateness. I have also updated the translated content, so this change does not cause todos for translators.

On my phone, but can someone look up and mention the existing issue that this fixes to close it? thank you!

On my phone, but can someone look up and mention the existing issue that this fixes to close it? thank you!

It's this one here: https://github.com/withastro/docs/issues/612
I've updated my PR description so that it auto-closes the issue.


Multiple windows

Is your feature request related to a problem? Please describe.
I'm always frustrated when I have to use VSCode as well as Lapce because I can't have multiple Lapce windows.
Describe the solution you'd like
The ability to have multiple Lapce windows concurrently.
Describe alternatives you've considered
Plugins: I do not know enough Rust to make one myself and I cannot find an existing plugin for it.
My friend managed to open another window via the command line, but I think this is unnecessarily complicated. Maybe add an option to add another instance to the system tray or right click on taskbar.
Additional context
Windows 11, Lapce 0.2.7

F1 > New window

In VSCode, my custom is passing Shift + CMD + N to create New Window, and now I have set the keybinding for this "New window" command is good.
but a simple issue is when in MacOS, some customer not really conveniently to pass F1, and the Menu there is no this command item expose.
shall we add "New Window" at Menu in file nav ?

As a new Windows Lapce user, I was very surprised not to be able to open a second window when I run lapce.exe again. When trying to start Lapce a second time, two window frames appear briefly and it looks like the process for the second instance crashes without an error message. It took me a little while to find the solution in the form of this post (thx panekj). The onboarding for a new user could be simplified a bit in my opinion.


<EMAIL_ADDRESS>does not work with nuxt bridge

Reproduction
https://codesandbox.io/s/agitated-cdn-cszk6?file=/package.json
Steps to reproduce the behavior

Go to reproduction link
Rebuild sandbox
See error

Expected behavior
Application must build without errors
Actual behavior
Application is build with errors
Additional information
After updating these dependencies
@pinia/nuxt  ^0.1.6  →  ^0.1.7
pinia        ^2.0.6  →  ^2.0.7

the following error occurred

P.S<EMAIL_ADDRESS>and<EMAIL_ADDRESS>work without errors

This is really weird as that release shouldn't contain any changes. Maybe a bug with one of the dependencies

@posva thank you!


Update docs about azure-spring-boot-starter-keyvault-certificates

https://docs.microsoft.com/en-us/azure/developer/java/spring-framework/configure-spring-boot-starter-java-app-with-azure-key-vault-certificates

https://github.com/MicrosoftDocs/azure-dev-docs/pull/552

Closing this PR because related PR has been merged: https://github.com/MicrosoftDocs/azure-dev-docs-pr/pull/2011


Stronger guarantees for atomics

I'm fairly sure that what we're doing with Relaxed ordering isn't correct; not 100% sure that what I'm doing here is accurate, but it should at the very least race less. Thanks!

I don't think there is any practical race condition. First, in the specific case of tide: we are never actually accessing this across threads. It only needs to be send and sync in order to work around the awkwardness of tide's Ext not getting passed from request to response. We have to take a clone, but there is only ever one "active" clone. Second, and less importantly for tide, it doesn't matter what order two threads write true into this atomic. But there will never be multi thread access, so that's irrelevant

I don't think there is any practical race condition.

Yeah, you're not wrong. I guess I'm more worried about the: "are all guarantees of this crate upheld at the crate boundary". And I think the answer is: "no". We implement Clone + Send + Sync for this type, which means that if it's used outside of Tide it can definitely be shared between threads and it very much can trigger a data race. This also means that if we ever change the exact way how Tide is used, we might trip over this too
The perf cost of atomics in this case worries me less than potentially needing to debug a race condition caused by this a good while later. A bug of this form could very well take the form of: "when running a Tide benchmark on a MacBook built after 2020 and session middleware is enabled, we observe race conditions that shoulnd't be possible" and that sounds like a bug that would be rough to properly track and fix 😅

This crate is certainly used outside tide. It would be nice to see this PR merged.


Add some more detail on how to use AppScope in AWS Lambda

@mQkatz would like your feedback please.

LGTM.

@iapaddler: 3rd environment variable added, please re-check.

@iapaddler All today's additions look good to me. Can you please confirm @bdalpe's change and (if appropriate) approve and merge?


A new chapter: Hive v4

Dear Hive Community,
I'd like to apologize for my lack of support and updates for Hive over the past year. Despite this, your commitment to the community has ensured that the package is still in a remarkably good state, and for that, I'm incredibly grateful.
As many of you know, I have spent a lot of time optimizing Isar to be as fast, reliable, and fun to use as possible. However, I recognize that a lot of you truly appreciate the simplicity of Hive. The thought of discontinuing Hive did cross my mind, but I now see a much more constructive way forward:
Over the next several weeks and months, I plan to invest my energy into the development of Hive v4. The primary focus will be to maintain the user-friendly interface and feature set that you've come to love. However, instead of continuing with the custom storage engine that has been causing several issues, we will transition Hive's data storage to Isar.
This significant change comes with several exciting advantages, including support for multiple isolates, significantly improving resource efficiency and reducing startup time, among others. Moreover, this will simplify the maintenance of Hive, ensuring its longevity through sustained support.
I cannot thank you enough for your continued support and use of Hive. Together, let's continue to make it a tool we all love and rely on. 💜
Simon

Great news!
I love Isar, but I also still use Hive in the same project, because it's just a simple key-value storage.
I use it to store cache, counters, json responses, lists and maps. I don't need complex stuff like indexes and queries, because I already have that data in isar.

Hey Simon, thanks for an update! I am wondering what's your plans regarding storage migration. How do you think to handle the transition to make sure existing applications won't lose any data.
Also considering both hive and isar has (or had) their own way of encoding data types, not sure about isar, but hive also have support for writing custom encoders/serializers. What's the plan for migrating them?

I am wondering what's your plans regarding storage migration.

That's indeed the biggest problem to solve. I think the best approach for now is to let the hive generator generate migration code.
For custom encoders & decoders I didn't find a solution yet. Maybe we can just deprecate them and ask users who rely on them to stay on Hive v2

Are you planning to support database queries like Isar does? If so, then it's though.
If not (only key to value lookups), maybe we can continue using the existing hive generated (or custom) encoders to encode structs into byte stream and store it within isar with its respective key.

This is really exciting news!

Are you planning to support database queries like Isar does? If so, then it's tough, I don't have an idea to create smooth migration plan.

In my opinion, a migration to Isar as the data store should focus solely on that and leave out advanced functionality that Isar could provide. That simplifies the migration and maintains the simplicity of Hive. If you need more advanced functionality, using Isar directly is likely a better fit.

Are you planning to support database queries
In my opinion, a migration to Isar as the data store should focus solely on that

Yeah agreed! I think has a good feature set and if someone is looking for queries, Isar should be used directly. The only feature I think would be valuable for Hive is (optional) atomic transactions.

I merged my work in the main branch. Check it out and provide feedback please 💜

Hey @simc , that plan sounds great!
I wanted to use Isar but decided to stick with Hive, as Hive supports Flutter on the web. With the migration to Isar as the engine for Hive, will it still work over IndexedDB?

This is promising, but I have a few questions.
Is there a list of things that Hive v4 will lack vs. v2?
Will there be a v2 support branch if v4 will not have feature parity?
Also what of Hive v3, is it being abandoned?

With the migration to Isar as the engine for Hive, will it still work over IndexedDB?

No. Isar web uses SQLite compiled to WASM and so will Hive.

Is there a list of things that Hive v4 will lack

I don't think there is anything Hive v4 will lack. The API is almost identical with a few minor differences.

Also what of Hive v3, is it being abandoned?

To my knowledge Hive v3 was never released as a stable version and I don't have time to maintain it unfortunately.

Does "No more code generation" mean that we don't need to reference a generated file like part 'main.g.dart';?
If so sounds pretty good.  As a follow-on, does this also hold for Isar 4.0?

@simc, thanks for reply. Can you confirm that given this issue: https://github.com/isar/isar/issues/1362 (Isar on the web supports only in-memory sqlite), in practice, Hive v4 cannot be used as a persistency service on the web?

Not right now that's correct @t-kozak. I hope to implement that before a stable release

Because MY_APP depends on hive ^4.0.0 which doesn't match any versions, version solving failed.
You can try the following suggestion to make the pubspec resolve:

Consider downgrading your constraint on hive: flutter pub add hive:^2.2.3

@pishguy You can use hive: ^4.0.0-dev.2 to get access to the pre-release :)

Hi @simc, first of all thanks for your work! I guess that the listenable method from version 2 (hive_flutter) has been replaced by watchKey and can now be used with a StreamBuilder.
I would like to ask what you think about the idea of making watchKey generic to minimize type casts if values of different data types are stored inside the box? For example: watchKey<bool?>(key) that returns a Stream<bool?>.
Also, I remember that in version 2 the value stored in the box could be read and used directly in the ValueListenableBuilder on the first build. In version 4 the value is always initially null. While I could give the StreamBuilder an initialValue using box.get(key), that seems a bit cumbersome. Do you plan to have the stream return such an initial value?
Thanks in advance! :)

Hi @simc,
First of all, I want to express my sincere appreciation for all your hard work on Hive and the dedication you've shown to the community. Hive has been an invaluable tool for me, and I'm excited to see its future unfold with Hive v4.
I'm sure many of us are eager to get our hands on the new Hive v4. Do you have a rough estimate of when we can expect a release candidate or a beta version?
Lastly, cheers from Munich! 🍻
Looking forward to Hive's continued evolution and success.

@simc Do you have any rough time schedule when a stable release for Hive v4 will be available?

Please make it happen i can't wait to see Hive V4.0.0 official release

@simc Do you have any rough time schedule when a stable release for Hive v4 will be available?

At the same week US starts using metric system 👀.

LGTM!

Hi @simc,
I tried the v4 pre release version(^4.0.0-dev.2) and its absolutely awesome, registering the type with its .fromJson method is such an elegant way.
I really want to use it on my app and just wondering when will there be stable release.
If its still got time can you maybe specify how unstable it is? I would love to switch to that for an Android app I'm working on.
Anyway thanks for making this amazing package.

I have a PR built on Hive v2 that supports Flutter web WASM if anyone wants to try it: https://github.com/isar/hive/pull/1291
I know it's not really relevant to this discussion, but I figured people with notifications turned on for this issue might be interested.

flutter: 3.19.6
hive: ^4.0.0-dev.2
isar_flutter_libs: ^4.0.0-dev.14
path_provider: ^2.1.0
static Box  settingBox = Hive.box(name: "setting", maxSizeMiB: 128);

When I call settingBox.clear();, the data is indeed cleared, and after restarting the app, there is no data. However, in the sandbox, the .isar size is 20MB, and I noticed that almost all .isar files are 20MB in size, but I'm sure the data they store is less than 1MB. Why is this happening?
some error
"DbError (12): Cannot allocate memory"
I am sure that the phone has enough RAM and enough disk storage space.
carsh in:
case ERROR_DB_FULL:
       throw DatabaseFullError();
     default:
       final length = IsarCore.b.isar_get_error(IsarCore.stringPtrPtr);
       final ptr = IsarCore.stringPtr;
       if (length != 0 && !ptr.isNull) {
         final error = utf8.decode(ptr.asU8List(length));
         throw DatabaseError(error);

Any updates V4 stable release?

For anyone still on v2, I released hive_ce that supports WASM compilation and Sets. Also the hive_ce_generator generates a HiveRegistrar extension so you can register all your adapters in one call (and not forget to register new ones like I used to every single time):
import 'package:your_package/hive_registrar.g.dart';

void main() {
  final path = Directory.current.path;
  Hive
    ..init(path)
    ..registerAdapters();
}

Issue: Box Name Causing Non-Responsiveness
Description
When using Hive.box(name: boxName) in Hive 4.x.x, the application stops responding after the Isar.open function call if the boxName is set to "test5". However, if the boxName is set to "1" (still a String), everything works as expected.
Steps to Reproduce
~Initialize Hive.
~Use Hive.box(name: "test5").
~Call Isar.open.
~The application becomes non-responsive.
Expected Behavior
The application should not become non-responsive regardless of the box name provided.
Actual Behavior
When the boxName is set to "test5", the application stops responding after Isar.open. Changing the boxName to "1" or anything resolves the issue.
Additional Details
When using 'test5' as the boxName, Isar.fastHash(typeName) returns a typeId such as -90198......, as shown in the instanceId in the second screenshot.
The application does not proceed beyond the point shown in the second screenshot.
Anything other than 'test5' for the boxName works fine, but 'test5' does not.
The issue might be unique to my setup, but I spent several hours trying to identify this problem.
environment:
  sdk: '>=3.4.4 <4.0.0'
  flutter: ">=1.17.0"

dependencies:
  flutter:
    sdk: flutter
  hive: ^4.0.0-dev.2
  isar_flutter_libs: ^4.0.0-dev.13
  logger: ^2.4.0
  path_provider: ^2.1.0

dev_dependencies:
  flutter_lints: ^4.0.0
  flutter_test:
    sdk: flutter
  very_good_analysis: ^6.0.0

[✓] Flutter (Channel stable, 3.22.3, on macOS 14.5 23F79 darwin-arm64, locale tr-TR)
[!] Android toolchain - develop for Android devices (Android SDK version 35.0.0-rc4)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.1)
[✓] VS Code (version 1.92.0)
[✓] Connected device (4 available)
[✓] Network resources```

hive_flutter: ^1.1.0
hive: ^2.2.3
Hive is not persisting data after flutter mobile app gets terminated.. Is there any solution for this ??

hive_flutter: ^1.1.0 hive: ^2.2.3
Hive is not persisting data after flutter mobile app gets terminated.. Is there any solution for this ?? @simc

Flutter secured storage

I switched to objectbox recently.

I just use hive_ce since that was easier than migrating to an entirely different storage solution

I just use hive_ce since that was easier than migrating to an entirely different storage solution
Actually, it's not different, it's just a package with a different name that contains traces of each version of Hive.
I recommend the Hive 4.x.x and my Hive extension https://pub.dev/packages/bee_hive

I'm the maintainer of hive_ce so I know what it is. I have many apps that are using Hive v2 in production, and I see no easy way to migrate to v4. Even if there was a way to migrate from v2 to v4, I have little to no faith in the future support of Hive. Please do not recommend using Hive v4 in production applications. I wouldn't even recommend using hive_ce for new projects unless you intend to contribute.
I am aware of Hive v4 not requiring code generation, but you lose the efficient binary storage of Hive v2 which was like... the entire point of choosing Hive over other options...

I'm the maintainer of hive_ce so I know what it is. I have many apps that are using Hive v2 in production, and I see no easy way to migrate to v4. Even if there was a way to migrate from v2 to v4, I have little to no faith in the future support of Hive. Please do not recommend using Hive v4 in production applications. I wouldn't even recommend using hive_ce for new projects unless you intend to contribute.
I am aware of Hive v4 not requiring code generation, but you lose the efficient binary storage of Hive v2 which was like... the entire point of choosing Hive over other options...

Actually, I'm not fully aware of the advantages of binary storage. But based on my research, I believe the new Hive may be faster than the old one.

I switched to objectbox recently.

do you face any issue while application move to terminate state?? on restart does it restore previous save data ??

I just use hive_ce since that was easier than migrating to an entirely different storage solution

do it keep you data saved when mobile app gets terminated or crash ??

do you face any issue while application move to terminate state?? on restart does it restore previous save data ??

@appconsultio Sure it does, and tbh i would be suprised if hive wouldn't do that too. Are you sure you're doing everything correctly on app startup?

I'm still using Hive and is working fine after phone restart or crash, the only place I have issues is on web.
@Rexios80  at this point I'm considering moving away from hive. do hive_ce has any improvement for web? I love the binary storage, but if the phone is a budget phone and you have stored a massive Box the app could suffer opening if you open that box in launch.

@oscarshaitan
hive_ce supports Flutter web WASM compilation which drastically improves performance.
If you're having issues opening boxes on startup, does making them lazy help?

hive_flutter: ^1.1.0 hive: ^2.2.3
Hive is not persisting data after flutter mobile app gets terminated.. Is there any solution for this ?? @simc

@simc plz reply how to solve this issue ??

@oscarshaitan hive_ce supports Flutter web WASM compilation which drastically improves performance.
If you're having issues opening boxes on startup, does making them lazy help?

Yes Im thinking on moving some to lazy.
I'm going to join to your discord so I can make more questions there

@simc Thank you for all the great work. Any idea when we can expect a release version of Hive v4 based on Isar?
Would it be possible to execute queries on a Hive box using Isar?

I believe @simc has stopped supporting this package. The sad truth of the Open-Source Community is that we do not get paid, and hence the wobbly support for packages.

RIP peace dear Bees

Can someone suggest alternative since the V4 is dead now.

Can someone suggest alternative since the V4 is dead now.

I use Hive Community Edition:
https://pub.dev/packages/hive_ce

Can someone suggest alternative since the V4 is dead now.

I use Hive Community Edition: https://pub.dev/packages/hive_ce

I love the way Hive V4 does, code-gen make the folder a bit messy tbh

@chunghei0116
I'm the maintainer of hive_ce
Doesn't Hive v4 still need code-gen from json_serializable to work?
Also I'm making code-gen much better in this PR. With that change, you only need one generated adapters file per project. You can see how it works in the updated README. I'm waiting on feedback from users to release the feature. If you could try it out and put any feedback in a comment on that PR that would be very helpful!

@Rexios80
Thank you for your contribution! I’ll definitely give it a try in my new project. Since I’m currently working on an enterprise project, I need a stable and well-maintained package. Cheers!

@chunghei0116
I just benchmarked Hive v2 vs v4 and the results are... not looking good...

Here's the PR for the benchmark if you want to check it out: https://github.com/IO-Design-Team/hive_ce/pull/43


react-native-curved-bottom-bar doesn't seem to be linked.make sure:

Error: The package 'react-native-curved-bottom-bar' doesn't seem to be linked. Make sure:

You rebuilt the app after installing the package
You are not using Expo Go

Having same error using expo

same error here after upgrade expo and react native

Hi all,
What is the version of this library you are using?

@hoaphantn7604
I am facing issue 3.1.0 version.
But if i use below commend it will working fine. npm i<EMAIL_ADDRESS>
Okay! I’m working on it.

hi All,
This issue is resolved. Please install new version.
Refer this example https://github.com/hoaphantn7604/react-native-curved-bottom-bar#use-in-expo

@hoaphantn7604 Thank you so much


docs: Add docs for Relation literals

I've combined it with the from_text page, as I think they're similar (and I put these first since I think they're generally better).
One question — do we want any keyword for them — e.g. from_lit {...}? I don't think we necessarily do, but wanted to raise it, since it's the only construction which can be directly in line now.

If you want, you can use:
from {[a = 3]}
select a

... because from is already just a noop function. Its only utility is "pulling from database tables" instead of normal name resolution.
I'd say let's not add too much unnecessary functions here.

My preference is starting with from. (The word "lit" does not seem very clear to me.)
Given things like from foo.csv and from df (df is pandas.DataFame on Python) in DuckDB, I wouldn't mind putting something other than table name after from.

My preference is starting with from. (The word "lit" does not seem very clear to me.)
Given things like from foo.csv and from df (df is pandas.DataFame on Python) in DuckDB, I wouldn't mind putting something other than table name after from.

This resonates with me...

Merging this; with a note on us considering from...


Buttons seem to be broken in the latest version of Chrome

I went to the demo site and attempted to click on the info icon and nothing happened.
It works in IE, not sure why it's not working, there were no errors in the Chrome's console.

The demo is outdated and runs an old version Lychee. The bug has been fixed already, but thanks for letting me know! https://github.com/electerious/Lychee/issues/345


Increase type safety

What Changed
Added some types and removed obsolete "any" type assertions by replacing them through stronger types
Change Type
Indicate the type of change your pull request is:

[x] maintenance
[ ] documentation
[ ] patch
[ ] minor
[ ] major

📦 Published PR as canary version: 0.5.9--canary.89.4c1ac9e.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

Hey @valentinpalkovic thank you so much for this contribution, I love this!!
There are still a couple issues in this PR, could you check them? It happens when running yarn update-all:
yarn update-all
yarn run v1.22.17
$ yarn update-configs && yarn update-docs
$ ts-node ./tools/update-configs
/Users/yannbraga/open-source/eslint-plugin-storybook/node_modules/ts-node/src/index.ts:750
    return new TSError(diagnosticText, diagnosticCodes);
           ^
TSError: ⨯ Unable to compile TypeScript:
tools/update-configs.ts:8:28 - error TS7016: Could not find a declaration file for module '../.prettierrc'. '/Users/yannbraga/open-source/eslint-plugin-storybook/.prettierrc.js' implicitly has an 'any' type.

8 import prettierConfig from '../.prettierrc'
                             ~~~~~~~~~~~~~~~~
tools/update-configs.ts:10:16 - error TS2665: Invalid module name in augmentation. Module '../.prettierrc' resolves to an untyped module at '/Users/yannbraga/open-source/eslint-plugin-storybook/.prettierrc.js', which cannot be augmented.

10 declare module '../.prettierrc' {}
                  ~~~~~~~~~~~~~~~~
tools/update-configs.ts:27:7 - error TS7053: Element implicitly has an 'any' type because expression of type 'string' can't be used to index type '{ 'import/no-anonymous-default-export': string; }'.
  No index signature with a parameter of type 'string' was found on type '{ 'import/no-anonymous-default-export': string; }'.

27       setting[rule.ruleId] = rule.meta.docs.recommended || 'error'
         ~~~~~~~~~~~~~~~~~~~~
tools/update-configs.ts:43:29 - error TS7053: Element implicitly has an 'any' type because expression of type 'string' can't be used to index type '{ csf: null; recommended: null; 'csf-strict': string; }'.
  No index signature with a parameter of type 'string' was found on type '{ csf: null; recommended: null; 'csf-strict': string; }'.

43   const extendsCategoryId = extendsCategories[category.categoryId]

Hey @yannbf
I will take care of it directly after The Easter holidays. :) Have a nice holiday time!

Thank you so much! I wish you an incredible holiday!!

@yannbf Please take another look.
It seems that ts-node does a different kind of type checking in the background than tsc --noEmit does. It complains about strict-mode specific restrictions, although it shouldn't complain. Therefore, I have disabled type-checking for ts-node and introduced a further step in the CI pipeline to check Typescript types explicitly via tsc.

@yannbf Could you take another look? :)

Hey @yannbf thank you for the review!
If everything looks fine, should I merge it? :) I'm just asking because normally I haven't triggered any merges in storybookjs repos so far and I want to make sure if it is ok to do so.


Failed pip install -r requirements.txt, rasa train, rasa shell

After pip install -r requirements.txt
ERROR: Cannot install -r requirements.txt (line 6) and rasa[spacy]==3.1.0 because these package versions have conflicting dependencies.

The conflict is caused by:
    rasa-sdk 3.1.0 depends on Sanic-Cors<2.0.0 and >=1.0.0
    rasa[spacy] 3.1.0 depends on sanic-cors<3.0.0 and >=2.0.0

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/user_guide/#fixing-conflicting-dependencies

After rasa train, I’m getting Core training was skipped because no valid domain file was found. Only an NLU-model was created. Please specify a valid domain using the '--domain' argument or check if the provided domain file exists.
After rasa shell --debug, and i say hi, gives sklearn.exceptions.NotFittedError: Vocabulary not fitted or provided
If this is really some version error from bumping 2.8.9 to 3.1.0, why does the domain file not found and sklearn's NotFittedError occur?

Thanks for raising this issue, @camattin will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗


mail dovecot: auth: Fatal: sql: driver not set in configuration file

Hello, I've been trying to run the mailserver, but after many retries I keep ending up having the following error :
mailserver      | 2017-05-30T15:39:19.929715+00:00 mail dovecot: auth: Fatal: sql: driver not set in configuration file /etc/dovecot/dovecot-sql.conf.ext
mailserver      | 2017-05-30T15:39:19.931327+00:00 mail dovecot: master: Error: service(auth): command startup failed, throttling for 60 secs

I repeats itself over and over every 30~60 seconds, PostfixAdmin and RainLoop interfaces are showing in the browser, but impossible to connect to the service (via Thunderbird or RainLoop, howerver I'm getting a prompt response via telnet on ports 993 and 587). I followed every step of the README, except for the optional NSD section that I've commented in docker-compose.yml and DKIM entries that I omitted from the DNS configuration (I'm not familiar with this extra security stuff, I'd like to enable it later when the basic services work).
Am I getting something or is there a bug somewhere?
Thank you very much,
Camille.

Try this command : docker exec -ti cat /etc/dovecot/dovecot-sql.conf.ext and post the result here please.

Hi thank you for your reply. Here is the result of the command, but I guess that some container name is missing in the args or something like that:
# docker exec -ti cat /etc/dovecot/dovecot-sql.conf.ext
Error response from daemon: No such container: cat

Thanks,
Camille.

I think this was the intended command, it seems that the file is empty:
# docker exec -ti mailserver cat /etc/dovecot/dovecot-sql.conf.ext
#

If I try with another file path it says No such file or directory so I guess the file is indeed here, but empty…

I have tried to install mailserver several times from scratch, using the following commands to purge the previous installations:
docker rm -f $(docker ps -a -q)
docker rmi -f $(docker images -q)
docker rm -v -f $(docker ps -a -q)
docker volume rm $(docker volume ls -q)

I always get the same dovecot-sql.conf.ext error. But early install problems was memory limitation, so I increased the RAM wich solved problem, but maybe some corrupted files of first intallation remain somewhere and are disrupting the new installation process…?

Try to build yourself :
docker rmi hardware/mailserver
docker build --no-cache -t hardware/mailserver https://github.com/hardware/mailserver.git#master
docker-compose up -d

Same error with the self build :-/ dovecot-sql.conf.ext still empty… What ca I do ? Thanks!

docker info

Here it is:
Containers: 5
 Running: 5
 Paused: 0
 Stopped: 0
Images: 15
Server Version: 17.03.1-ce
Storage Driver: devicemapper
 Pool Name: docker-202:1-160624-pool
 Pool Blocksize: 65.54 kB
 Base Device Size: 10.74 GB
 Backing Filesystem: ext4
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 2.087 GB
 Data Space Total: 107.4 GB
 Data Space Available: 2.272 GB
 Metadata Space Used: 2.834 MB
 Metadata Space Total: 2.147 GB
 Metadata Space Available: 2.145 GB
 Thin Pool Minimum Free Space: 10.74 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: false
 Deferred Deletion Enabled: false
 Deferred Deleted Device Count: 0
 Data loop file: /var/lib/docker/devicemapper/devicemapper/data
 WARNING: Usage of loopback devices is strongly discouraged for production use. Use `--storage-opt dm.thinpooldev` to specify a custom block storage device.
 Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.90 (2014-09-01)
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 4ab9917febca54791c5f071a9d1f404867857fcc
runc version: 54296cf40ad8143b62dbcaa1d90e520a2136ddfe
init version: 949e6fa
Kernel Version: 3.10.105-xenU-34-082da8f-x86_64
Operating System: Debian GNU/Linux 8 (jessie)
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.954 GiB
Name: yooook
ID: YLBO:D4KA:IVIR:NUKB:PIYQ:HGZ4:SZH2:QUBZ:VW5E:ISOI:3WXS:HPQR
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

Thank you,
Camille.

I have not tested hardware/mailserver with the devicemapper storage driver.
Officially, this docker image has been tested with AUFS and OverlayFS storage drivers.
https://docs.docker.com/engine/userguide/storagedriver/selectadriver/
https://docs.docker.com/engine/userguide/storagedriver/aufs-driver/
https://docs.docker.com/engine/userguide/storagedriver/overlayfs-driver/

I just tried devicemapper driver on Archlinux with Docker 17.05.0-ce, it works well. Can you describe your host environement, your server setup. You said you increased memory so I guess you're in a virtualized environment.

Thanks for your reply. Yes my host is a VM at https://www.gandi.net/hosting/iaas it's an old installation, about 4 years, but I've upgraded it regularly, it works well. Now Debian GNU/Linux 8.8 (jessie) with a  3.10.105-xenU-34-x86_64 Linux Kernel. There are very few basic LAMP services (I already have an Apache so I reconfigured mailserver's NGnix to ports 8001 and 4431, directly accessible as well as via Apache reverse proxy on ports 80 and 443). There is also a very old debian that I need to keep running for some services in a chroot. I installed docker from apt repositories, the kernel is compiled by the PaaS, known to run docker properly: https://wiki.gandi.net/en/iaas/references/server/docker That's all I think about as useful information. What else could I tell? Does it raise some flag for you?
Thank you,
Camille.

If you can, can you try with OverlayFS as described here : https://wiki.gandi.net/en/iaas/references/server/docker
Just in case.

Hi, sorry for the delay, OverlayFS wasn't available on my kernel, I had to migrate the whole VM to HVM to get a newer kernel, new configuration broke the network so it took time to get it back up. I now have OverlayFS:
# docker info | grep Storage
Storage Driver: overlay

But still the same SQL problem… :-/
What can I do?
Thank you,
Camille.

Does the container have any mounts that shadow the original file?
docker inspect -f "{{json .Mounts}}" mailserver

Which image is the container based on?
docker ps | grep "CONTAINER\|mailserver"

Which id does the image have?
docker images | grep -e "REPOSITORY\|mailserver"

Hi, thanks for your reply, here are the outputs:
# docker inspect -f "{{json .Mounts}}" mailserver
[{"Type":"bind","Source":"/mnt/docker/mail/opendkim","Destination":"/etc/opendkim/keys","Mode":"rw","RW":true,"Propagation":""},{"Type":"bind","Source":"/mnt/docker/mail","Destination":"/var/mail","Mode":"rw","RW":true,"Propagation":""},{"Type":"volume","Name":"9c7ac7dbda3a8c1f9a7b3fda4f86330e02201ffb247aab2e2aed159610fe422f","Source":"/var/lib/docker/volumes/9c7ac7dbda3a8c1f9a7b3fda4f86330e02201ffb247aab2e2aed159610fe422f/_data","Destination":"/etc/letsencrypt","Driver":"local","Mode":"","RW":true,"Propagation":""}]

# docker ps | grep "CONTAINER\|mailserver"
CONTAINER ID        IMAGE                     COMMAND                  CREATED             STATUS              PORTS                                                                                NAMES
8d3304e66572        hardware/mailserver       "tini -- startup"        43 seconds ago      Up 42 seconds       <IP_ADDRESS>:25->25/tcp, <IP_ADDRESS>:143->143/tcp, <IP_ADDRESS>:587->587/tcp, <IP_ADDRESS>:993->993/tcp, <IP_ADDRESS>:4190->4190/tcp, 465/tcp   mailserver

# docker images | grep -e "REPOSITORY\|mailserver"
REPOSITORY                TAG                 IMAGE ID            CREATED             SIZE
hardware/mailserver       latest              09e33a1281f6        2 hours ago         236 MB

Thanks,
Camille.

Check the file attributes with ls -l
docker exec -ti mailserver ls -l /etc/dovecot/dovecot-sql.conf.ext

And verify the content of several other files, it may not be the only one in this case.

Thanks,
Camille.

Wich other files should I check?

Oh, I see envtpl is crashing because of Python UTF-8/unicode problem… I saw it first time, but I thought it was just a warning, but in fact it crashes…

Ok i see the problem. Avoid Unicode characters in all environment variables.
To solve your problem, checks the value of these variables and remove any unicode characters :
DBHOST, DBPORT, DBNAME, DBUSER, DBPASS

https://github.com/andreasjansson/envtpl/issues/20#issuecomment-306052408
Thanks for reporting this.

Thanks! It worked! :-) Bad characters were in DBPASS, BTW it seems to dislike # as well. Also I had a bug building the git version of hardware/mailserver:
Installing collected packages: envtpl, Jinja2, MarkupSafe
  Running setup.py install for envtpl

    Installing envtpl script to /usr/local/bin
Compiling /tmp/pip-build-bkv1Bj/Jinja2/jinja2/asyncfilters.py ...
  File "/tmp/pip-build-bkv1Bj/Jinja2/jinja2/asyncfilters.py", line 7
    async def auto_to_seq(value):
            ^
SyntaxError: invalid syntax

Compiling /tmp/pip-build-bkv1Bj/Jinja2/jinja2/asyncsupport.py ...
  File "/tmp/pip-build-bkv1Bj/Jinja2/jinja2/asyncsupport.py", line 22
    async def concat_async(async_gen):
            ^
SyntaxError: invalid syntax

I had to pull the ready made image. It worked.
So I'll finally give a try to NSD, but can I use the same key for multiple domains? I see /mnt/docker/mail/opendkim/ lists only the initial domain despite the fact that I add new ones on postfixadmin and rainloop?
Thanks!
Camille.

So I'll finally give a try to NSD, but can I use the same key for multiple domains? I see /mnt/docker/mail/opendkim/ lists only the initial domain despite the fact that I add new ones on postfixadmin and rainloop?

Use ADD_DOMAINS environment variable.

My bad, RTFM. Thanks a lot!
Camille.


feat: revert slippage tolerance in provide liquidity

Summary of changes

Report of required housekeeping

[ ] Github issue OR spec proposal link
[ ] Wrote tests
[ ] Added a relevant changelog entry to CHANGELOG.md

(FOR ADMIN) Before merging

[ ] Added appropriate labels to PR
[ ] Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
[ ] Confirm added tests are consistent with the intended behavior of changes
[ ] Ensure all tests pass

Codecov Report
Patch coverage: 96.66% and project coverage change: -0.06 :warning:

Comparison is base (a7b5748) 87.22% compared to head (cf75ab7) 87.16%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
- Coverage   87.22%   87.16%   -0.06%
==========================================
  Files          13       13
  Lines        2411     2431      +20
==========================================
+ Hits         2103     2119      +16
- Misses        308      312       +4

Impacted Files
Coverage Δ

contracts/starfleit_pair/src/testing.rs
99.43% <95.83%> (-0.13%)
:arrow_down:

contracts/starfleit_pair/src/contract.rs
92.68% <100.00%> (-0.25%)
:arrow_down:

... and 4 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

// then it must accept 1:1 and treat left amount as donation

In the test code, this part of the comment should be changed. It's testing different cases.

284b097


JWT Setup for Authentication

What github issue is this PR for, if any?
Resolves #1 #2
What changed, and why?
Set up JWT with Devise.
I had to extend Devise sessions controller's create action to return a custom response for JSON requests.
I also extend respond_to_on_destroy method so sign out responds in json.
Additionally, I had to modify the accessible module so JSON requests don't get redirected to the root path.
How is this tested? (please write tests!) 💖💪
I created request specs for good/bad login to verify correct headers were sent (e.x authorization, content-type...etc) and also JSON data.
I also made specs for the sign out route -> testing JWT is moved to denylist and a 200 response.
Postman was used to make changes (i.e unassign a case assignment) in the portal provided a JWT was present.
I also did a live test where I ran the rails server on a ngrok tunnel and frontend app using expo go side by side.
See screenshots below.
Screenshots please :)
Live test using Ngrok and my android phone running the frontend with Expo go.

https://github.com/ctc-casa-ios/api/assets/86758440/1c15c9fa-a6df-4450-bf8e-496df232d859
Postman: Success login means auth JWT in header

Postman: Failed login means auth JWT not in header

Postman: Unassign a case assignment using auth token passed in request header

Postman: Unassign a case assignment with no auth token

Postman: Sign out returns 200 response + JWT is moved to denylist

Feelings gif (optional)
What gif best describes your feeling working on this issue? https://giphy.com/
How to embed:

Feedback please? (optional)
We are very interested in your feedback! Please give us some :) https://forms.gle/1D5ACNgTs2u9gSdh9

I think this is good to go

we also need to ask casa how to store credentials.


feat(repeat): add repeat feature

Should be done

@lazav94 Thanks so much for the contribution, merging.


Basic integration testing

It would be good to have some level of integration testing for all the adapters.

@stevedomin I'd gladly take on this. Currently I'm writing integration tests using Hound, and I can't really tell if the message was sent or now.
I think we'd need something similar to ActionMailer in Rails, where the test adapter would store all of the send emails in memory, temporarily, so we can fetch those and check content etc.
In addition to that we'd need some sort of cleaning mechanism, so we can clean the sent out mail between tests.
Does that sound right for

@hubertlepicki this is actually for having some kind of integration testing for Swoosh itself (PR in progress => https://github.com/swoosh/swoosh/pull/57).
I do want to support what you're suggesting though (ran into this issue myself), I've opened #66 so we can discuss about it. Definitely open to you tackling it!


Ctrl-K works, Ctrl-V doesn't

Description of the problem or steps to reproduce

Ctrl-K - shows "Cut line" and removes line
Ctrl-V - shows "Pasted clipboard" and nothing else happens

Specifications
Commit hash: 3afb3d0
OS: Fedora 24
Terminal: ROXTerm 3.3.2 on Wayland

Does CtrlC work? And do you have xclip or xsel installed?

Works for me on Fedora 24 with xfce4-terminal. However, I have xsel installed.

$ xsel
bash: xsel: command not found...
$ xclip
Error: Can't open display: wayland-0

Ctrl-C says "Copied selection", but nothing happens on Ctrl-V.

You do not have xsel and xclip apparently does not work with Wayland. Can you try to install xsel from repository?

Anyway, I thought micro is supposed to use its own clipboard implementation if xsel or xclip are not available, but only in one instance of the application (so you cannot paste in different apps, but still can in micro)?

Oh, xsel will not probably work under Wayland either, as it is app for working with X.

I asked about wayland at http://stackoverflow.com/questions/39616066/wayland-clipboard-api and having private clipboard for micro is a no-go. Having multiple clipboards is no-go and it is really confusing thing in Linux.

IMHO Wayland is still not ready for prime time for various reasons and micro cannot do anything about it until Wayland matures much, much more.

Ok, it seems like this is a problem with xclip and wayland? If you want micro to use its own internal clipboard, then you should uninstall xclip and/or xsel.

wayland doesn't have some standardized protocol for doing this, the clipboard is implemented by the compositor like GNOME or something.

It seems like this is a specific problem with xclip/xsel not working properly in Wayland.


Add GL headers to compile on Windows

Those new types like PFNGLCREATESHADERPROC were giving syntax errors. Found glcorearb.h with those types. This header includes khrplatform.h so I had to add it too.

Oh wow, didn't know that windows by default does not usually have glext.h. That's a very useful information! Thanks for letting me know! I'll get back to this later.

@kolumb what do you think about c443baec1d1d890d68e3843c6c50c1d38677e982 ? Please let me know if there are any problems on Windows with that.

@rexim Superb, it works!

@kolumb nice! Thank you for your contribution!


More performant conversion byte to hexadecimal string

I am closing this merge request, because it incorrectly accepts some non-ascii characters as hex characters.
I have added new test cases to the main branch, to help detect this problem.


Bind error

With the last major version (2.x.x), the error property in bind funtion was more complete, why not in 3?
For example, the error 49 has suberrors, I would like to know the suberrors with version 3.
I refer to this: https://community.bmc.com/s/article/How-to-understand-LDAP-Error-Codes
For example, error code 49, error subcode 525, 52e, etc.

Can you please provide examples of error messages you are talking about? I do not understand the issue.

Sorry, I think it is not an error, or maybe yes.
v2.x.x.
bind error response:
LDAPError [InvalidCredentialsError]: 80090308: LdapErr: DSID-0C090447, comment: AcceptSecurityContext error, data 533, v3839
v3.x.x.
bind error response
LDAPError [InvalidCredentialsError]: Invalid Credentials
As you can check, the error description in v3 is less complete than v2.
Thanks.

Please provide the full stack trace for the errors instead of just the message property.

v3
LDAPError [InvalidCredentialsError]: Invalid Credentials
at messageCallback (C:\Users\username\Desktop\my-project\node_modules\ldapjs\lib\client\client.js:1261:45)
at Parser.onMessage (C:\Users\username\Desktop\my-project\node_modules\ldapjs\lib\client\client.js:919:14)
at Parser.emit (node:events:527:28)
at Parser.write (C:\Users\username\Desktop\my-project\node_modules\ldapjs\lib\messages\parser.js:135:8)
at TLSSocket.onData (C:\Users\username\Desktop\my-project\node_modules\ldapjs\lib\client\client.js:875:22)
at TLSSocket.emit (node:events:527:28)
at addChunk (node:internal/streams/readable:315:12)
at readableAddChunk (node:internal/streams/readable:289:9)
at TLSSocket.Readable.push (node:internal/streams/readable:228:10)
at TLSWrap.onStreamRead (node:internal/stream_base_commons:190:23) {
lde_message: 'Invalid Credentials',
lde_dn: null
}
v2
LDAPError [InvalidCredentialsError]: 80090308: LdapErr: DSID-0C090447, comment: AcceptSecurityContext error, data 533, v3839
at messageCallback (C:\Users\username\Desktop\my-project\node_modules\ldapjs\lib\client\client.js:1220:45)
at Parser.onMessage (C:\Users\username\Desktop\my-project\node_modules\ldapjs\lib\client\client.js:888:14)
at Parser.emit (node:events:527:28)
at Parser.write (C:\Users\username\Desktop\my-project\node_modules\ldapjs\lib\messages\parser.js:107:8)
at TLSSocket.onData (C:\Users\username\Desktop\my-project\node_modules\ldapjs\lib\client\client.js:875:22)
at TLSSocket.emit (node:events:527:28)
at addChunk (node:internal/streams/readable:315:12)
at readableAddChunk (node:internal/streams/readable:289:9)
at TLSSocket.Readable.push (node:internal/streams/readable:228:10)
at TLSWrap.onStreamRead (node:internal/stream_base_commons:190:23) {
lde_message: '80090308: LdapErr: DSID-0C090447, comment: AcceptSecurityContext error, data 533, v3839\x00',
lde_dn: null
}

I also met this error code: InvalidCredentialsError today, What is the meaning of this code and under what circumstances may it appear

Please file a different issue if you think binding is not working correctly. This issue is about the information in the error.

Regarding this actual issue: I am not ignoring it. To diagnose how the difference occurs will require running through a test case on both v2 and v3 with a debugger to inspect what happens where. It will take me some time to get to it. If anyone wants to provide such an analysis, that'd be great.

This is really important.
v2.3.x:  InvalidcredentialsError: 80090308: LdapErr: DSID-OC090447, comment: AcceptSecurityContext error, data 532...
v3.0.3: {Invalid Credentials} (code: 49, name: InvalidCredentialsError)
data 532 - means that the password is expired, while code 49 gives no clues what's wrong.
I am afraid but I think we will need to downgrade as it is very difficult to troubleshoot errors in v3.

It seems it isn't important enough for you to contribute anything useful to the conversation nor a solution.

As soon as we get more data I will share our analysis. We modified our code to be able to use v2 and v3 based on an environment variable. Invalid credentials error looks the same on both versions. Unfortunately we can't test right now with expired password error, where we saw the difference in the past.


ParanoidBoolean causes issues with belongs_to Relationships

class Car
include DataMapper::Resource
has n, :wheels
property :id, Serial
property :deleted, ParanoidBoolean, :default => false, :lazy => false
end
If :lazy is set to true, Wheel.car will return nil even if a record exists (it even queries the DB successfully).
See https://gist.github.com/1511964 for a fuller use-case

To follow up: This only presents with existing records pulled from the datastore.  Creating new records will result in correctly formed relationships.


[FEATURE] Creating a Tic-Tac-Toe game in CPP

Detailed description
This is a Tic-Tac-Toe game in C++.
Context
This is for the coders/programmers who want to implement Tic-Tac-Toe in C++.
Possible implementation
No response
Additional information
No response

can u assign me this issue

can you assign this to me.
Here’s a simplified  description for implementing Tic-Tac-Toe in C++:
Create a 3x3 board using a 2D array to represent the game grid.
Display the board after each move so players can see the current game state.
Take input from the players (Player 1 = 'X', Player 2 = 'O') for selecting a position on the grid.
Check for a win after each move by verifying if three symbols are aligned in a row, column, or diagonally.
Check for a draw if all positions are filled and no one has won.
Alternate turns between Player 1 and Player 2 until there is a winner or a draw.
Display the result and offer the option to restart the game or exit.

i have updated it

Hi @rogerdemello !! Can you assign me this issue?

@AK261102 i want to work on this can you assign me  this

tag him bro  @rogerdemello

can u assign me this issue

I have already solved it, can you please assign me this  issue

hey @rogerdemello can you assign me this