ComodoCA testcase fails Cert revoked on jdk-11+

https://ci.adoptopenjdk.net/job/Grinder/7590/console
Testcase failing on jdk11+: security/infra/java/security/cert/CertPathValidator/certification/ComodoCA.java
11:26:49  Received exception: java.security.cert.CertPathValidatorException: Certificate has been revoked, reason: UNSPECIFIED, revocation date: Tue Mar 02 04:51:39 CST 2021, authority: CN=COMODO RSA Extended Validation Secure Server CA, O=COMODO CA Limited, L=Salford, ST=Greater Manchester, C=GB, extension OIDs: []

Ref: https://github.com/eclipse/openj9/issues/12176
But now confirmed affects Hotspot as well, and can see Cert has been revoked...

Upstream problem: https://bugs.openjdk.java.net/browse/JDK-8263059


Darkmode button design

Resolves #98 and
Resolves #99

Changes:

Better design for darkmode/lightmode button (icons: sun/moon)
added animation when clicking on the button => changes styles

Changes 2:

also fixed the bug, when pressing on upload, that all inputs were shifted to the left side

fixed it, please review if this was what you mean


add missing response body to blocked requests

Description
Adds the missing response body to blocked requests.
Examples
{
    "error":{
        "code":"deniedByPolicy",
        "message":"Operation denied due to security policies",
        "innererror":{
            "date":"2023-05-10T19:52:19Z",
            "filename":"policies.yaml",
            "method":"POST",
            "path":"/remote.php/dav/spaces/storage-users-1$some-admin-user-id-0000-000000000000",
            "request-id":"5e55a8ad-cd5f-40c9-a002-db9e30f24585"
        }
    }
}

{
    "error":{
        "code":"deniedByPolicy",
        "message":"Operation denied due to security policies",
        "innererror":{
            "date":"2023-05-10T20:05:00Z",
            "filename":"Neue Datei.md",
            "method":"PUT",
            "path":"/remote.php/dav/spaces/storage-users-1$some-admin-user-id-0000-000000000000/Neue Datei.md",
            "request-id":"9302a92e-c7eb-4c41-9a98-62eb37cf5db4"
        }
    }
}

Related Issue

Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

test environment:
test case 1:
test case 2:
...

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[ ] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

I don't think we want to have request denied at top level. This makes it hard to have a genetic error handler

@michaelstingl This is a proposal to have a unified response body for all requests which are blocked by the policy engine to enable all clients to distinguish them. In a second step and in the scope of a cross platform initiative we should introduce server translated error messages.

I don't think we want to have request denied at top level. This makes it hard to have a generic error handler.
Otherwise I like it

use error ?

I wouldn't nest it personally, because the http status code indicates the error already, but that's just personal preference. Error as top level is fine for me as well.

Error as top level is fine for me as well.

It is the same like libregraph. So keeping error at the top level makes it consistent.

💪

Anyways: I would start the message with capital letter. In the end this is what the user reads (en).

Ok. Maybe a misunderstanding on my side. I thought we agree on a fixed string for now which will be translated in web.

Sure sure sure. But let's start this thing of correct and give it a handy message from the beginning, so that an English user gets a cool message.
We will translate it in web but when we change that in the future, you don't need to replace the English strings later ;)

Ok, tried to give it a meaningful and short message.
Operation denied due to security policies
@tbsbdr @kulmann

:boom: Acceptance test localApiTests-apiAsyncUpload-ocis failed. Further test are cancelled...

:boom: Acceptance test localApiTests-apiCors-ocis failed. Further test are cancelled...


MINOR: Delete config/kraft/README.me

It's out of data and we have since updated the main documentation to cover KRaft.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@ijuma , we've just merged the updated readme for kraft: https://github.com/apache/kafka/pull/12642 . I don't think this is out of date now.

@showuon Yes, I know, but I still think we should delete it since we now have the documentation in the right place. This was a placeholder until we had that.

@conradlco , nice find. Are you interested in submitting a PR to fix it?

Since I broke this, I went ahead and submitted a PR https://github.com/apache/kafka/pull/12750
Thanks for flagging this @conradlco .


Run Code Analysis in the lab only on debug

We are hitting errors in the build lab due to Code Analysis failing spuriously on code in the Arm build.
We had originally set up Code Analysis to only run in the lab on debug builds to reduce the impact on the build time. A recent change enabled in unconditionally. Restoring the conditional logic, which will hopefully solve the build lab issue.
Test CI build running here:
https://dev.azure.com/ms/microsoft-ui-xaml/_build/results?buildId=93193&view=results

/azp run


griddler.io is outdated

We can clean up the wording, copyright date, and who to tweet at (at the bottom).

Fixed in #193.


Problem when compiling with nmake W10.

I've trying to compile libSVM on W10 with Visual Studio 2015 Enterprise compiler and I get this error.
I type  nmake -f makefile.win
Microsoft (R) Program Maintenance Utility Version 14.00.24210.0
Copyright (C) Microsoft Corporation.  All rights reserved.
    cl.exe -nologo -O2 -EHsc -I. -D __WIN32__ -D _CRT_SECURE_NO_DEPRECATE -c timer.cpp

timer.cpp
timer.cpp(26): fatal error C1083: Cannot open include file: 'limits': No such file or directory
NMAKE : fatal error U1077: '"c:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\bin\cl.exe"' : return code '0x2'
Stop.
It seems some problem with visual studio installation?

but there is no timer.cpp in libsvm?
Reinier César Mujica Hernández writes:

I've trying to compile libSVM on W10 with Visual Studio 2015
Enterprise compiler and I get this error.
I type nmake -f makefile.win
Microsoft (R) Program Maintenance Utility Version 14.00.24210.0
Copyright (C) Microsoft Corporation. All rights reserved.
cl.exe -nologo -O2 -EHsc -I. -D __WIN32__ -D _CRT_SECURE_NO_DEPRECATE -c timer.cpp

timer.cpp
timer.cpp(26): fatal error C1083: Cannot open include file:
'limits': No such file or directory
NMAKE : fatal error U1077: '"c:\Program Files (x86)\Microsoft Visual
Studio 14.0\VC\bin\cl.exe"' : return code '0x2'
Stop.
It seems some problem with visual studio installation?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
*


Integrated power on collection surface

It would be useful for demo/ITER to predict the power incident on machine surfaces. Could be implemented as a large pixel, or collection of large pixels with cosine weighted hemispherical sampling.

This was completed with the observer refactoring. New Observer framework enables this functionality.


[style]: Can we change the position of theme button on different media

What's the style idea?
We need to improve the theme button position on different media
In desktop size, we should adjust the position of the theme button around the search bar.
on the mobile view that will be the same
Add screenshots
Add screenshots
Code of Conduct

[X] I agree to follow this project's Code of Conduct

hey @amanpoddar-dev12 i want to contribute in this, can you assign this to me

📢 Invitation to Join the DevDisplay WhatsApp Community!
We are excited to invite you to join the WhatsApp community of DevDisplay, where developers and contributors come together to collaborate and share their ideas.
Collaborative Support: If you are interested in working together with extensive support, we encourage you to join our community.
Open Source Contribution: If you would like to contribute to the DevDisplay open-source project, your participation is welcomed in our WhatsApp group.
By joining, you will have the opportunity to connect with like-minded individuals, discuss project developments, and contribute to our shared goals.
For more information and to join the community, please visit: Join DevDisplay WhatsApp Community.
We look forward to your participation!


Youtube reloads when I receive a call

This addon works perfectly for my Android Firefox browser. Except when I receive a call when a youtube video is playing in the background, the video page gets reloaded as soon as I hang up the call and i lose my watch progress.

That's sounds more like an issue with either Firefox and/or Android. (I assume the video/audio still gets stopped when you receive the call, right?)

Yes, video/audio get stopped when i receive calls.

If the video gets stopped by Firefox due to something else (i.e. in this case an incoming call) requesting audio focus, that's outside of the control of what the add-on does. And the fact that the page subsequently gets reloaded is probably some variation of https://bugzilla.mozilla.org/show_bug.cgi?id=1807364, which again is a Firefox and/or Android issue, and nothing the add-on can do anything about.

Cheers, that actually makes sense to know that it's a firefox bug for sure. Let's hope they fix it someday. :)


Invalid deserialisation order for required fields

Source JSON
{}

Destination type
public class SampleType
{
	[JsonProperty(Required = Required.Always)]
	public int SampleField { get; }

	[JsonConstructor]
	public SampleType(int sampleField)
	{
		if (sampleField == 0) {
			throw new Exception("Should not be zero");
		}
		SampleField = sampleField;
	}
}

Expected behavior
Deserialization exception is thrown:

Required property 'SampleField' not found in JSON. Path '', line 1, position 2.

Actual behavior
User exception is thrown:

Should not be zero

Steps to reproduce
JsonConvert.DeserializeObject<SampleType>("{}");

Hello?..

er, have you looked at the test suite? I personally would not expect it to be able to call that ctor as it does not have a sampleField - perhaps if it was Nullable ?

What test suite you are telling about? I'm just calling raw Json.Net api. And sure it can find and call constructor, you can just copypaste the example and see it yourself.

TL; DR - I'm pretty sure this is not how JsonConstructor works - the inputs need to be present or the ctor params need to be Nullable (i.e. ?)
I'm talking about the test suite in this repo - if I need to figure out how something is expected to work, I cut out the middleman and read the tests to determine how such cases are handled. Consider them an alternate form of documentation.
In your particular case, you're saying "this case is wrong". I'm saying "go look at the other cases and see if you can find something similar; it's likely that there are specs saying the exact opposite based on the reason I am suggesting".
My reasoning for this is that a) you want it to work that way b) it doesn't so c) someoene will need to make it do what you want and not contravene existing rules so d) it might as well be you that has a quick scan to determine whether your request is realizable (or whether there's a different way to manage this)

TL; DR - I'm pretty sure this is not how JsonConstructor works - the inputs need to be present or the ctor params need to be Nullable (i.e. ?)

I expect if parameter is marked as non-nullable but missing in the scheme then serialized should throw "field not found" or somerhing. I don't expect field to be null, I want a proper exception when it happens.

My reasoning for this is that a) you want it to work that way b) it doesn't so c) someoene will need to make it do what you want and not contravene existing rules so d) it might as well be you that has a quick scan to determine whether your request is realizable (or whether there's a different way to manage this)

I expect scheme validation happening before deserializing. I think it's reasonable to expect it working that way.

Apologies - seems I scanned too fast - I'm surprised it entered the constructor as you're saying; I agree there is a case to be understood. But, the truth re these edge cases will definitely be in the tests...

Json.NET seems to be working as designed here.  Two points:

When deserializing using a parameterized constructor, Json.NET will automatically provide a default value for any constructor arguments without matching JSON properties.
This is described in, e.g. this answer to Newtonsoft json deserialise missing int values as nulls instead of zero and this answer to How does JSON deserialization in C# work.

When the c# data model has required properties and one or more are missing, then a JsonSerializationException is thrown after deserialization is complete.
When deserializing an object with a default constructor this falls out naturally from Json.NET's streaming approach to deserialization.  First the object is constructed.  Next Json.NET streams through the JSON, deserializing and populating each property and tracking which ones were found.  Finally, if a required property was not encountered, an exception is thrown at the end of the process.
It seems as though Newtonsoft made a conscious decision to preserve this order when deserializing objects with parameterized constructors.  They did this despite the fact that it would have been possible to throw the exception at the beginning of deserialization, since the JSON gets preloaded and deserialized into property and argument values before the object is constructed.
To confirm this, see JsonSerializerInternalReader.CreateObjectUsingCreatorWithParameters().  Checking for missing properties is done in the call to EndProcessProperty() which happens is towards the end of the algorithm, just before calling finally OnDeserialized.

If I modify your type as follows, so that validation of sampleField is done in an OnDeserialized method and a factory method that checks for appropriate argument values replaces the public parameterized constructor:
public class SampleType
{
    int sampleField;

    [JsonProperty(Required = Required.Always)]
    public int SampleField { get { return sampleField; } }

    public static SampleType Create(int sampleField)
    {
        Validate(sampleField);
        return new SampleType(sampleField);
    }

    [JsonConstructor]
    internal SampleType(int sampleField)
    {
        this.sampleField = sampleField;
    }

    [System.Runtime.Serialization.OnDeserialized]
    void OnDeserializedMethod(System.Runtime.Serialization.StreamingContext context)
    {
        Validate(SampleField);
    }

    static void Validate(int sampleField)
    {
        if (sampleField == 0)
        {
            throw new Exception("Should not be zero");
        }
    }
}

Then a JsonSerializationException is thrown as expected, rather than the custom exception.  Demo fiddle here.
I cannot, however, find anything in the official documentation that explains exactly how JSON properties are matched to constructor arguments, what happens when a constructor argument is missing, how and whether [JsonProperty] attributes applied to .Net properties affect the deserialization of similarly named constructor arguments, or when missing property validation occurrs.  Those are all topics that Newtonsoft might want to clarify in their documentation.

@dbc2 thanks for detailed explanation, it really makes things clear.
The main purpose of the code written was reusability. I mean this constructor can be used in both (de)serialization and regular class creation. So having all validations in constructor instead of magically decorated methods are much more appropriate.
I agree that docs may be clearer here but I also think that constructor should be called after fields are populated. It calls constructor with actual values in the end so it may already validate that some values are missing at this point. I don't think it sets private fields via reflection at this point, it should merely call the constructor with values. So, being said, values are already deserialized at this point.


Substitute ID references in href attributes

Initial PR #577

This is wrong @Kreeg. Needs a proper rebase

@XhmikosR yeah I kinda freaked out too when I saw that :D


Add itest for multi unit alerts

This PR adds an itest that addresses https://github.com/canonical/prometheus-k8s-operator/pull/239:
Now that juju_unit is no longer added to rule files, this test confirms that alerts still fire per unit (not only from the leader unit), and are correctly labeled as such.

Red tests are red

No pebble socket. Possible juju failure. Rerunning

Red tests are red

This passed locally but now here fails for
  Full diff:
  - ['avalanche/0', 'avalanche/1']
  + []

i.e. seems like no juju_unit labels on the alerts, in contrast to expectation. Investigating.

This test is flaky because the charm I uploaded to edge has a stray rule file:
https://github.com/canonical/charmcraft/issues/710


Display user identity in drawer when logged in

So that you know who you are logged in as

Won't be useful for most users.


math/cmplx: Tan incorrect on s390x for large arguments

s390x fails the tests for Tan with large arguments due to incorrect argument reduction in the trigonometric routines for the base math packages.
--- FAIL: TestTan (0.00s)
    cmath_test.go:839: Tan((-1.329227995784916e+36+0i)) = (NaN+NaNi), want (0.40806638884180424+0i)
    cmath_test.go:839: Tan((1.7668470647783843e+72+0i)) = (NaN+NaNi), want (-0.37603456702698074+0i)
    cmath_test.go:839: Tan((2.037035976334486e+90+0i)) = (NaN+NaNi), want (4.60901287677811+0i)
    cmath_test.go:839: Tan((-3.1217485503159922e+144+0i)) = (NaN+NaNi), want (3.391359650547799+0i)
    cmath_test.go:839: Tan((1.8919697882131776e+69+0i)) = (NaN+NaNi), want (-6.76813854009065+0i)
    cmath_test.go:839: Tan((-2.514859209672214e+105+0i)) = (NaN+NaNi), want (-0.7641769501660493+0i)

This is due to the fix for #31566, but the underlying issue has been present since #6794 was fixed.  The temporary solution after that change was to just disable the tests for s390x.
I think the s390x assembly implementations need to be fixed or they should just revert to the pure go versions for large arguments.
@griesemer @mundaym

Found a fix and made code changes. Tested the fix and made sure that its working fine. I will raise my code changes for review in Gerrit...

CC @golang/s390x

I have raised a CL to fix this issue and its under review. Please note that my fix has been tested and its working fine.
Please find the link below:
https://go-review.googlesource.com/c/go/+/470595
Benchmark details of Tan function with table driven inputs are as below:

Benchmarks
Without any code   changes:
With assembly   code changes
With native Go   code changes

Tan/2^(-1)-8
10.17 ns/op
10.23 ns/op
13.53 ns/op

Tan/2^(1)-8
10.02 ns/op
10.16 ns/op
13.16 ns/op

Tan/2^(10)-8
9.852 ns/op
10.11 ns/op
13.32 ns/op

Tan/2^(20)-8
10.62 ns/op
10.69 ns/op
12.77 ns/op

Tan/2^(40)-8
10.25 ns/op
10.76 ns/op
13.42 ns/op

Tan/2^(50)-8
10.69 ns/op
10.89 ns/op
13.16 ns/op

Tan/2^(51)-8
10.38 ns/op
10.52 ns/op
13.55 ns/op

Tan/2^(52)-8
2.505 ns/op
31.63 ns/op
40.26 ns/op

Tan/2^(60)-8
2.507 ns/op
33.48 ns/op
40.60 ns/op

Tan/2^(80)-8
2.520 ns/op
31.57 ns/op
42.28 ns/op

Tan/2^(100)-8
2.503 ns/op
31.94 ns/op
40.70 ns/op

Tan/2^(200)-8
2.617 ns/op
27.24 ns/op
37.57 ns/op

Tan/2^(400)-8
2.809 ns/op
28.51 ns/op
37.43 ns/op

Tan/2^(-10)-8
10.13 ns/op
10.48 ns/op
13.25 ns/op

Tan/2^(-15)-8
10.17 ns/op
10.29 ns/op
12.52 ns/op

Tan/2^(-20)-8
10.26 ns/op
10.53 ns/op
13.54 ns/op

Tan/3^(-10)-8
10.07 ns/op
10.18 ns/op
12.25 ns/op

Tan/3^(-15)-8
10.14 ns/op
10.31 ns/op
12.51 ns/op

From the above table, please note that the time taken for all the inputs beyond "2^51" till "2^400" is just around "2ns" (as shown in the first column) with the existing code, because the Tan function was only returning a "NaN" value as soon as the control enters it. No logic was getting executed and hence no time was spent in the function.


feat: plugin changeConfig hook

Usage:
import { Plugin } from 'vite'

export const changeConfigPlugin1: Plugin = {
  changeConfig: (prev) => {
    prev.define['__changeConfigPlugin1__'] = 'success'
    prev.plugins.push(changeConfigPlugin2)
    return prev
  }
}

const changeConfigPlugin2: Plugin = {
  changeConfig: (prev) => {
    prev.define['__changeConfigPlugin2__'] = 'success'
    return prev
  }
}

fix: https://github.com/vitejs/vite/issues/1162

I wonder if we should even let it return another config? It's faster to mutate than copy, but maybe no big deal?

This hook already allow you to return another config. Checkout this line: https://github.com/vitejs/vite/pull/1178/files#diff-e1f89aee8992fa7022cc43adb93042f9bbb4b223f0d6cd0e70719cf4b795afa4R164

And maybe prepareConfig has a nicer ring to it?

What does "ring" mean? Does it mean "name"?  :)
This hook is imitating rollup's options hook. How about name it options?

Can we move this into resolveConfig so it gets cached by #740?

I voted against that pr, checkout https://github.com/vitejs/vite/pull/740#issuecomment-736480556

Can we move this into resolveConfig so it gets cached by #740?

I voted against that pr, checkout #740 (comment)

Okay, but we should still move it into resolveConfig, I think.

And maybe prepareConfig has a nicer ring to it?

What does "ring" mean? Does it mean "name"? :)

https://www.gymglish.com/en/gymglish/english-translation/a-nice-ring-to-it

This hook is imitating rollup's options hook. How about name it options?

I'm not a big fan of using "config" and "options" interchangeably. I think we should stick with "config" naming with a verb like "change" or "prepare" before it. Just naming it config might give the wrong impression.

I wonder if we should even let it return another config? It's faster to mutate than copy, but maybe no big deal?

This hook already allow you to return another config. Checkout this line: #1178 (files)

Sorry, I worded that confusingly. I meant we should consider not allowing a new config to be returned. Two reasons to do this: (1) so other plugins can keep a stable reference to the Vite config, and (2) it's at least slightly more performant to mutate the config than to copy it.


Notebooks: Only see SQL kernel in dropdown

SQL is the only kernel that shows up using the latest bits from master:

I can't say with 100% certainty that it's the VS Code merge... But it's fishy...

The problem here does not repro 100%, and I can no longer repro it unfortunately...
The problem was that the NotebookService wasn't getting created before the notebook provider registration was occurring:
(from notebookserviceimpl.ts):
this._register(notebookRegistry.onNewRegistration(this.updateRegisteredProviders, this));
(onNewRegistration was firing before this listener was hooked up).
Let's see if we can repro this again. If we can, then we should probably check the registry one more time when getting a list of notebook providers instead of relying solely on this event firing in the right order.

@chlafreniere, after Anthony rollback, i don't see this issue anymore


Ipsec routers containers are failing in HA setup on Azure

Rancher Versions:
Server: 1.4.1
Docker Version:
1.12.6
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 14.04
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
HA Rancher on Azure, the setup consists of:
(443) Application Gateway --> (80) Classic azure load balancer --> (8080) 3 Rancher servers
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:

Deploy the HA setup mentioned above on azure
Try to register any host to the setup

Results:
The ipsec-router container is always failing with the following logs:
3/28/2017 12:38:34 AM+ export CHARON_PID_FILE=/var/run/charon.pid
3/28/2017 12:38:34 AM+ CHARON_PID_FILE=/var/run/charon.pid
3/28/2017 12:38:34 AM+ rm -f /var/run/charon.pid
3/28/2017 12:38:34 AM+ export PIDFILE=/var/run/rancher-net.pid
3/28/2017 12:38:34 AM+ PIDFILE=/var/run/rancher-net.pid
3/28/2017 12:38:34 AM+ GCM=false
3/28/2017 12:38:34 AM+ (( i=0 ))
3/28/2017 12:38:34 AM+ (( i<6 ))
3/28/2017 12:38:34 AM+ ip xfrm state add src <IP_ADDRESS> dst <IP_ADDRESS> spi 42 proto esp mode tunnel aead 'rfc4106(gcm(aes))' 0x0000000000000000000000000000000000000001 128 sel src <IP_ADDRESS> dst <IP_ADDRESS>
3/28/2017 12:38:34 AM+ GCM=true
3/28/2017 12:38:34 AM+ ip xfrm state del src <IP_ADDRESS> dst <IP_ADDRESS> spi 42 proto esp
3/28/2017 12:38:34 AM+ break
3/28/2017 12:38:34 AM+ '[' false == true ']'
3/28/2017 12:38:34 AM+ DEBUG=
3/28/2017 12:38:34 AM+ mkdir -p /etc/ipsec
3/28/2017 12:38:34 AM+ curl -f -u 20C1D6EF3CC459628C2F:wrxmvk1UawM1a7thXu54DAgfJSNQGXZMqbdxJnjW https://hgalalazure.rancher.space/v1/configcontent/psk
3/28/2017 12:38:34 AM  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
3/28/2017 12:38:34 AM                                 Dload  Upload   Total   Spent    Left  Speed
3/28/2017 12:38:34 AM
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100    32  100    32    0     0     79      0 --:--:-- --:--:-- --:--:--    79
3/28/2017 12:38:34 AM+ curl -f -X PUT -d '' -u 20C1D6EF3CC459628C2F:wrxmvk1UawM1a7thXu54DAgfJSNQGXZMqbdxJnjW 'https://hgalalazure.rancher.space/v1/configcontent/psk?version=latest'
3/28/2017 12:38:34 AM  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
3/28/2017 12:38:34 AM                                 Dload  Upload   Total   Spent    Left  Speed
3/28/2017 12:38:35 AM
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
3/28/2017 12:38:35 AM++ ip route get <IP_ADDRESS>
3/28/2017 12:38:35 AM++ awk '{print $3}'
3/28/2017 12:38:35 AM+ GATEWAY=<IP_ADDRESS>
3/28/2017 12:38:35 AM+ iptables -t nat -I POSTROUTING -o eth0 -s <IP_ADDRESS> -j MASQUERADE
3/28/2017 12:38:35 AM++ ip route get <IP_ADDRESS>
3/28/2017 12:38:35 AM++ grep via
3/28/2017 12:38:35 AM++ awk '{print $7}'
3/28/2017 12:38:35 AM+ exec rancher-net -i <IP_ADDRESS>/16 --pid-file /var/run/rancher-net.pid --gcm=true --use-metadata --charon-launch --ipsec-config /etc/ipsec
3/28/2017 12:38:35 AMtime="2017-03-27T21:38:35Z" level=info msg="Writing pid 1 to /var/run/rancher-net.pid"
3/28/2017 12:38:35 AMtime="2017-03-27T21:38:35Z" level=info msg="Using backend: ipsec"
3/28/2017 12:38:35 AMtime="2017-03-27T21:38:35Z" level=info msg="Using metadata: true"
3/28/2017 12:38:35 AMtime="2017-03-27T21:38:35Z" level=info msg="Reading info from metadata"
3/28/2017 12:38:35 AMtime="2017-03-27T21:38:35Z" level=error msg="couldn't get self container from metadata: Error 404 accessing /self/container path"
3/28/2017 12:38:36 AMtime="2017-03-27T21:38:36Z" level=info msg="Listening on :8111"
3/28/2017 12:38:36 AMtime="2017-03-27T21:38:36Z" level=error msg="couldn't get self container from metadata: Error 404 accessing /self/container path"
3/28/2017 12:38:36 AMtime="2017-03-27T21:38:36Z" level=error msg="couldn't reload the overlay: Error 404 accessing /self/container path"
3/28/2017 12:38:36 AMtime="2017-03-27T21:38:36Z" level=fatal msg="Error 404 accessing /self/container path"
3/28/2017 12:38:36 AM00[DMN] Starting IKE charon daemon (strongSwan 5.5.0, Linux 3.13.0-107-generic, x86_64)
3/28/2017 12:38:36 AM00[CFG] loading ca certificates from '/usr/local/etc/ipsec.d/cacerts'
3/28/2017 12:38:36 AM00[LIB] opening directory '/usr/local/etc/ipsec.d/cacerts' failed: No such file or directory
3/28/2017 12:38:36 AM00[CFG]   reading directory failed
3/28/2017 12:38:36 AM00[CFG] loading aa certificates from '/usr/local/etc/ipsec.d/aacerts'
3/28/2017 12:38:36 AM00[LIB] opening directory '/usr/local/etc/ipsec.d/aacerts' failed: No such file or directory
3/28/2017 12:38:36 AM00[CFG]   reading directory failed
3/28/2017 12:38:36 AM00[CFG] loading ocsp signer certificates from '/usr/local/etc/ipsec.d/ocspcerts'
3/28/2017 12:38:36 AM00[LIB] opening directory '/usr/local/etc/ipsec.d/ocspcerts' failed: No such file or directory
3/28/2017 12:38:36 AM00[CFG]   reading directory failed
3/28/2017 12:38:36 AM00[CFG] loading attribute certificates from '/usr/local/etc/ipsec.d/acerts'

Network manager container log

3/27/2017 10:16:10 PMtime="2017-03-27T19:16:10Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"
3/27/2017 10:16:10 PMtime="2017-03-27T19:16:10Z" level=error msg="Failed to apply cni conf: Error: No such container: 27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611"
3/27/2017 10:16:10 PMtime="2017-03-27T19:16:10Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"
3/27/2017 10:16:10 PMtime="2017-03-27T19:16:10Z" level=error msg="Failed to apply cni conf: Error: No such container: 27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611"
3/27/2017 10:16:11 PMtime="2017-03-27T19:16:11Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"
3/27/2017 10:16:11 PMtime="2017-03-27T19:16:11Z" level=error msg="Failed to apply cni conf: Error: No such container: 27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611"
3/27/2017 10:16:12 PMtime="2017-03-27T19:16:12Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"
3/27/2017 10:16:12 PMtime="2017-03-27T19:16:12Z" level=error msg="Failed to apply cni conf: Error: No such container: 27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611"
3/27/2017 10:16:12 PMtime="2017-03-27T19:16:12Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"
3/27/2017 10:16:12 PMtime="2017-03-27T19:16:12Z" level=error msg="Failed to apply cni conf: Error: No such container: 27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611"
3/27/2017 10:16:13 PMtime="2017-03-27T19:16:13Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"
3/27/2017 10:16:13 PMtime="2017-03-27T19:16:13Z" level=error msg="Failed to apply cni conf: Error: No such container: 27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611"
3/27/2017 10:16:14 PMtime="2017-03-27T19:16:14Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"
3/27/2017 10:16:14 PMtime="2017-03-27T19:16:14Z" level=error msg="Failed to apply cni conf: Error: No such container: 27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611"
3/27/2017 10:16:15 PMtime="2017-03-27T19:16:15Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"
3/27/2017 10:16:15 PMtime="2017-03-27T19:16:15Z" level=error msg="Failed to apply cni conf: Error: No such container: 27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611"
3/27/2017 10:16:16 PMtime="2017-03-27T19:16:16Z" level=info msg="Setting up binaries for: map[rancher-bridge:27dbfba623765b5b9df2370c62ad2f2526ed28bd81742d457caed2337136f611]"

The root cause is that agent ip is not set per host because the cattle set DETECT_CATTLE_AGENT_IP to a format like ip:port. The root cause is that cattle looks for x-forwarded-forheader to set agent ip and somehow azure lb is returning ip:port in their header. @ibuildthecloud

Rancher server: v1.6.1-rc5
I replicated the same HA setup in Azure:
(443) Application Gateway --> (80) Classic azure load balancer --> (8080) 3 Rancher servers
and then registered Azure hosts with Rancher server, i can see all services in all infrastructure stacks up and running normally.


Fix/cognito username robustness

Proposed fix for #27

Sorry for the messy PR, including the master branch changes.. 🙈 But I hope this demonstrates the proposed fix for #27


new definition

This PR fixes...
Remove the example below. Then, add the related issue after the hash. Lastly, tell us what problems this PR is meant to solve.
Ex: This PR fixes #999. We didn't have a unicorn, so we went to the unicorn store and got one.
What I did...
Remove the example below. Then, list out the work you have done, in bullet form.
Ex:

one
two

How to test...
In a list format, tell us how to test this PR.
Ex:

Navigate to https://example.com/cats
Click on magic button
See coolest cat

I learned...
Did you run into any interesting issues while making this PR? If you learned anything you would like to share, we'd love to know! Even if it is a fun fact or GIF!

This is a new definition for the COT website.

@katiyelae great! Hit the 'edit' button to fill out the PR template with the necessary information:


feat(undeploy) - Add an option that, when true, will also remove buil…

…ds, buildConfigs and Imagestreams
fixes #190
nodeshift undeploy removeAll

Coverage increased (+0.06%) to 95.868% when pulling 63de6c0c5669f596fb52d7efdedcd37cca207a71 on lholmquist:190-unbuild into c381b9e7cc2a3850804d290436fdf32c08884c25 on bucharest-gold:master.

@lance @helio-frota so if we were to add this to rhoaster, undeploy(with the removeAll=true flag) should remove everything

@lholmquist sorry, I'm not 100% following the source code of rhoaster and nodeshift.
In some way this can help to have something more easy for rhoaster users to undeploy-or-not without need to comment the line  .then(_ => test.onFinish(testEnvironment.undeploy))  ?
If yes -> then LGTM
if no and if this ^ makes no sense, nevermind my comment :smiley:

@helio-frota currently when undeploy happens, it just removes the deployment, route, service stuff. this would allow someone to also remove the builds and imagestreams that were created,  totally cleaning up everything.


Increase column width when listing releases

(Second attempt with correct author.email)
After #1560 was merged, names of releases can now be up to 53 characters long, up from 14 previously. When listing releases, the maximum column width is 30 characters, meaning it's not possible to get the full name of releases using the full available length.
This change just increases the maximum column width to 60, to allow for longer release names.
PS.
I'm working on getting a CLA signed by my employer, even if it does seem a bit silly for such a small change. :)

This change is

Looks good. As soon as the LF CLA passes I will merge.

In theory, the CLA should now be in order. I guess it tries checking again after a comment..?

Or do I have to push a new commit?

Thank you! I'm merging this into 2.1 now.


patch before init-agent

In some case,  patch need to be applied before init-agent. since that uses WMCore code to create the db.

@amaltaro, Alan, please review

I don't remember if we have all the files in place before the init-agent (or is it activate-agent...).
If you don;t want to wait, just merge it. Otherwise I'll test it on Tuesday

and before you say 'stop working', I'm stopping now :-D


Initial open of Edit Pin screen takes quite long time

Problem is in navigation controller appearance settings. When setting barStyle or preferredStatusBarStyle, controller load takes random time to complete. Some times it freezes after did load another time after will appear.
I suppose to change design and get rid of navigation controller here.
logs:
tap edit pin
start open edit 0
controller created 0
did load 8862        !!!!!
will appear 6
did appear 561
tap edit pin
start open edit 0
controller created 0
did load 95
will appear 7
did appear 1366     !!!!!


Publish target not working with relative path

Using VS 2017 I created a new ASP.Net Core project (net461). I went to publish the project using "Web Deploy Package" option and chose a relative path ("..\output\Test.zip").
I get the following error.
C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\Sdks\Microsoft.NET.Sdk.Publish\build\netstandard1.0\PublishTargets\Microsoft.NET.Sdk.Publish.MSDeployPackage.targets(88,5): Error : Web deployment task failed. (The path '..\output\WebApplication2.Parameters.xml' is not valid.)
Package failed.
If I change the package path to be something like "C:\temp\Test.zip" it works fine and creates the package.

Thanks for the work around, I had the same issue.
Got some more detail which might be interesting. If you select a relative path it does:

create the folder
create *.Parameters.xml (with content)
create *.SourceManifest.xml (with content)
NOT create the zip

VS2017 Professional (15.2.26430.6)
ASP.NET Core  Web Application (.NET Framework) - WebAPI

I also have the same problem.
I also tried with:
Test.zip
.\Test.zip
./Test.zip
in all cases I get the same error as above.


Enable IP forwarding fo a VM's NIC using Bosh

Versions (please add them if you have more releases for this issue)
CPI      Version: any
BOSH     Version: any
CF       Version:
Diego    Version:
Stemcell
Version: any
Distro:

[x] Ubuntu
[x] CentOS

Environment

Azure Environment:

[x] Azure
[ ] AzureChinaCloud
[ ] AzureUSGovernment
[ ] AzureGermanCloud
[ ] AzureStack

Version: (e.g. 1712 update or build number 180106.1)

[ ] Others:

Location: (e.g. East US)
Subscription ID:
Resource Group Name:
[ ] Use Managed Disks

Issue Details
This is a feature request not a bug or issue with the CPI.
Currently, there is no way using the Azure Bosh CPI to enable IP forwarding in the IP configuration of the NIC for a machine created via Bosh. It would be great if you can support such a feature. There could be a cloud property ip_forwarding for example that can be added as vm extension in cloud-config and attached to an instance group. This will enable/disable IP forwarding on the nic of this VM.
This is currently supported in the CPI of GCP. You can find the documentation here:
https://bosh.io/docs/google-cpi.html#vm-types
It will be great if Azure CPI can offer simillar functionality.

Thanks @firasnajjar for raising the feature request. Just want to let you know that we will evaluate it and keep you posted.

Updates: We are working on the codes.
https://github.com/bingosummer/bosh-azure-cpi-release/commits/ip-forwarding


Token saved to OutstandingToken does not match custom serializer output

When a custom serializer adds extra fields to the JWT payload and the rest_framework_simplejwt.token_blacklist app has been added to INSTALLED_APPS in settings, the token field in OutstandingToken does not match that returned to the user.
The following 2 lines in the code are responsible for entering the string representation of the tokens into the database:
https://github.com/davesque/django-rest-framework-simplejwt/blob/master/rest_framework_simplejwt/tokens.py#L210
and
https://github.com/davesque/django-rest-framework-simplejwt/blob/master/rest_framework_simplejwt/tokens.py#L230
This does not directly influence any of the blacklisting functionality due to the fact that it uses the jti to decide if a token is valid, this is more a problem that could cause confusion due to the mismatch between the token in the database and the token given to the user.
So far I have not found a nice solution to this problem from an external stand point and it seems action will be required within this codebase to resolve this.
In my specific case I am only adding more user information to the JWT, so making the Token object swappable in django-rest-framework-simplejwt and overwriting the for_user method in my application could be sufficient. It may also remove the need for a custom serializer in this case.
Let me know your thoughts, I'm happy to contribute to the solution if needed.

I need a solution for that, I face the same problem, does anyone solved it ??


refactor: use treesitter functions from upstream

Drop (at least one) dependency on nvim-treesitter and simply use the vim.treesitter API provided by upstream Nvim.
Note that nvim-treesitter's ft_to_lang is already (informally) deprecated, and the long term roadmap of nvim-treesitter is to remove the module system and stop serving as a dependency for plugins, so a change like this will be needed eventually anyway.

Ty @gpanders! I can’t check ATM as traveling, which version added this API?

vim.treesitter.start() was added in 0.9.
If you'd like to support versions older than that, you could use
local ok, parser = pcall(vim.treesitter.get_parser, bufnr, ft)
if not ok then
  return false
end

vim.treesitter.highlighter.new(parser)
return true

instead (this is exactly what vim.treesitter.start does).

Ty @gpanders, will test and merge this PR if works as expected.

It requires that the parser for a filetype be installed, but that is also true when using nvim-treesitter, is it not?
As a test, I changed it locally to this:
local ts_attach = function(bufnr, ft)
  local ok, err = pcall(vim.treesitter.start, bufnr, ft)
  assert(ok, err)
  return true
end

For filetypes which I have a parser (C, Lua), there is no error and highlighting works. For filetypes which I do not have a parser (e.g. YAML), I see an error. But this is exactly what I would expect. If I don't have a parser installed it should fallback to regex syntax highlighting.

It requires that the parser for a filetype be installed, but that is also true when using nvim-treesitter, is it not?

That was my thought too, I have parser installed for all my files, for some odd reason the old code works but when using vim.treesitter.start it fails, since it works for you I’ll do some more digging.

I suggest keeping an eye on the main branch of nvim-treesitter, which will eventually become the default (or at least merged into master): https://github.com/nvim-treesitter/nvim-treesitter/tree/main
The code as-is will fail when using that branch because it removes ft_to_lang, which is why I looked into this issue and created this PR in the first place.
No immediate action is needed, but as I said it's worth keeping an eye on so that you can stay ahead of any breakages when that branch is eventually merged.

I suggest keeping an eye on the main branch of nvim-treesitter, which will eventually become the default (or at least merged into master): https://github.com/nvim-treesitter/nvim-treesitter/tree/main
The code as-is will fail when using that branch because it removes ft_to_lang, which is why I looked into this issue and created this PR in the first place.
No immediate action is needed, but as I said it's worth keeping an eye on so that you can stay ahead of any breakages when that branch is eventually merged.

Agreed, we can leave this PR open until we figure out the alternative, we can also monitor Telescope as they'd be facing the same issue.

https://github.com/ibhagwan/fzf-lua/commit/74e8200a350e1e6a4f3b3d3dbc7c87fc7ddaa2df
@gpanders, merged manually with backward compatibility (did not remove the 0.8 code) and a couple more changes, you can't use the ft directly with vim.treesitter.start as it doesn't always match, you first have to call vim.treesitter.language.get_lang(ft) to acquire the right ts language name.
The new implementation is:
https://github.com/ibhagwan/fzf-lua/blob/74e8200a350e1e6a4f3b3d3dbc7c87fc7ddaa2df/lua/fzf-lua/previewer/builtin.lua#L691-L703
You still get credit for the entire commit btw :-)


Add flake8 in CI

Description
Add linting checking with flake8.
Fixes #(issue)
Type of change
Please check options that are relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have read the contributing guidelines
[x] I have updated the HISTORY.rst and AUTHORS.rst files
[x] Linting passes successfully : flake8 . --exclude=doc
[x] Typing passes successfully : mypy mapie examples --strict
[x] Unit tests pass successfully : pytest -vs --doctest-modules mapie
[x] Coverage is 100% : pytest -vs --doctest-modules --cov-branch --cov=mapie --pyargs mapie
[x] Documentation builds successfully : cd doc; make clean; make html

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@9aa986e). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master       #63   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         9
  Lines             ?       554
  Branches          ?        42
==========================================
  Hits              ?       554
  Misses            ?         0
  Partials          ?         0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9aa986e...c7aa895. Read the comment docs.


Replacement for accessing return value of Xrm.Utility.openWebResource

This documentation says to use the Xrm.Navigation.openWebResource method as a replacement for Xrm.Utility.openWebResource, but it doesn't explain how to perform the replacement for code that uses the return value from Xrm.Utility.openWebResource, which is a Window object representing the window that is opened by Xrm.Utility.openWebResource. This is necessary for web resources that open other web resources, and that use window.postMessage for passing messages to each other.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 75ccf05e-86b9-3638-ccd6-e18849c82eb2
Version Independent ID: d550c9e5-4fc4-6ed7-e50c-93584ee7b7ca
Content: openWebResource (Client API reference) in Dynamics 365 Customer Engagement
Content Source: ce/developer/clientapi/reference/Xrm-Navigation/openWebResource.md
Service: crm-online
GitHub Login: @KumarVivek
Microsoft Alias: kvivek

I agree with @amervitz. Xrm.Navigation.openWebResource should return a window object. I need this to make sure I don't open the same window multiple times.

What is the status of this? Looks like no activity since Aug 2018?

@amervitz Yes, this is an issue and I have passed on the feedback to the product team. They now have a work item open to track/address it.
Closing the issue.


Modular Arithemetic queries on 10^500000 constraints

In this, we are given two variables a and b, will perform operations like additions, multiplication, divisions, subtraction, and power of that two numbers with modular operation having numbers as 1<a,b<10^50000. This is a tricky one but good to know this.
I am _ participant in GSSoC 2021_, Can @tarun26091999 you plz assign me this issue?

@tarun26091999  I made PR and also completed the requested changes, please take a look again.


Build an image running with Nuke gives TarException

Describe the bug
I use Nuke to automate a lot of tasks on my projects, like tests environment configuration, and I did not have any problem running containers with it.
But, when I try to build an image I'm getting:
Unhandled exception. ICSharpCode.SharpZipLib.Tar.TarException: Entry closed at '0' before the '3183' bytes specified in the header were written
      at ICSharpCode.SharpZipLib.Tar.TarOutputStream.CloseEntry()
      at ICSharpCode.SharpZipLib.Tar.TarOutputStream.Finish()
      at ICSharpCode.SharpZipLib.Tar.TarOutputStream.Dispose(Boolean disposing)
      at System.IO.Stream.Close()
      at System.IO.Stream.Dispose()
      at ICSharpCode.SharpZipLib.Tar.TarArchive.Dispose(Boolean disposing)
      at ICSharpCode.SharpZipLib.Tar.TarArchive.Dispose()
      at DotNet.Testcontainers.Images.DockerfileArchive.Tar()
      at DotNet.Testcontainers.Clients.DockerImageOperations.BuildAsync(IImageFromDockerfileConfiguration configuration, CancellationToken ct)
      at Program.<Main>$(String[] args) in C:\d\nuke-testcontainers\src\Program.cs:line 24
      at Program.<Main>(String[] args)

This happens if I run a test, csproj, or even if I put the build logic inside the Nuke Target.
If I run the logic outside Nuke everything works as expected
To Reproduce
Just try to build an image with/from a Nuke target
I made an example to demonstrate the bug:
https://github.com/lucasteles/nuke-testcontainers
If you run on the repository root:
dotnet run --project .\src\

it will work as normal
To get the error run on the repository root :
dotnet tool restore
dotnet nuke

Expected behavior
Should not throw an exception when running from Nuke

Is VS running? Exclude the .vs directory via .dockerignore, see (#609):
https://github.com/testcontainers/testcontainers-dotnet/blob/aba9f75020bcd7f309e905fe71c874a765919eef/examples/WeatherForecast/.dockerignore#L2
Or another process is blocking a file.

The .NET implementations use Glob pattern matching, it looks like there are more cases covered by Docker or Go. As far as I know, * will match everything except slashes. Files in .nuke/ are not ignored e.g.
Try the following:
**
!src
src/obj
src/bin

OC, * should behave the same like in Docker.

@HofmeisterAn changing the glob to ** worked 🎉

👍 I will reopen it. It makes sense to support * too (same behavior as Docker). Other developers may stumble across the same issue. We can map a single * to ** internal, that would fix the issue without much effort.


docs: Added MIT compliant LICENSE

Description
Added MIT compliant LICENSE
Type of Change

[X] Examples / docs / tutorials / dependencies update
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Security fix

Checklist

[X] I've read the CODE_OF_CONDUCT.md document.
[X] I've read the CONTRIBUTING.md guide.
[X] I've updated the code style using black.
[X] I've written tests for all new methods and classes that I created.
[X] I've written the docstring in numpy style for all the methods and classes that I used.

As raised here, #27, we should have an accompanying LICENSE file.

Thanks a lot for the quick answer @michealroberts!


fix internet ready check

Add internet ready check

Should this implement a StatusCallbackMap like other modules? Not sure if we use the alive checks for anything currently, but might be useful to be consistent with the other checks in the future


Wrong order calculation for multi-lines container

In case if not all lines are full filled up the Swiper generate wrong source number. For example you can try 16 items, slidesPerColumn: 3, slidesPerColumnFill: 'column' in case if 6 columns are visible. Proposed fix is (core.js, starting in line 515:)
        // Calc slides
        var slideSize;
        var slidesPerColumn = s.params.slidesPerColumn;
        var slidesPerRow = slidesNumberEvenToRows / slidesPerColumn;
        var numFullColumns = slidesPerRow - (s.params.slidesPerColumn * slidesPerRow - s.slides.length);
        for (i = 0; i < s.slides.length; i++) {
            slideSize = 0;
            var slide = s.slides.eq(i);
            if (s.params.slidesPerColumn > 1) {
                // Set slides order
                var newSlideOrderIndex;
                var column, row;
                if (s.params.slidesPerColumnFill === 'column') {
                    column = Math.floor(i / slidesPerColumn);
                    row = i - column * slidesPerColumn;
                    if (column > numFullColumns || (column==numFullColumns && row==slidesPerColumn-1)) {
                        if (++row>=slidesPerColumn) {
                            row=0;
                            column++;
                       }
                    }
                    newSlideOrderIndex = column + row * slidesPerRow;
                    slide
                        .css({
                            '-webkit-box-ordinal-group': newSlideOrderIndex,
                            '-moz-box-ordinal-group': newSlideOrderIndex,
                            '-ms-flex-order': newSlideOrderIndex,
                            '-webkit-order': newSlideOrderIndex,
                            'order': newSlideOrderIndex
                        });
                }
                else {
                    row = Math.floor(i / slidesPerRow);
                    column = i - row * slidesPerRow;

                }
                slide
                    .css({
                        'margin-top': (row !== 0 && s.params.spaceBetween) && (s.params.spaceBetween + 'px')
                    })
                    .attr('data-swiper-column', column)
                    .attr('data-swiper-row', row);

            }

....
Thank you,
-=A=-

Do you have a live example or JSFiddle so i can understand better this problem?

Just try to display 16 slides in 3 columns with filling up by 'column'. Last two columns should  contains 2 slides only (not 3). According to the current code source-orders will be :
0  1    2  3  4   5
6   7   8  9  10
12 13 14 15 16
As you can see source order 11 is missed and second row is not properly filled up. Proper source order should be
0    1   2  3   4   5
6    7   8  9  10 11
12 13 14 15

Unfortunately, this issue does not seem to be fixed completely:
Using the fill methods "column" or "row" leads to similar behaviour but it should not. See this fiddle with a 3x3 grid (3 slides per view, 3 slides per column), fill method "column" and 10 slides total. One would expect that the initial view has 9 slides visible, instead it has only 8.
It seems that this is due to the fill order which comes aparent when switching to a grid with 4 slides per view, 3 slides per column (forked fiddle):

Green arrows mark the expected fill order (consistent to the slide numbers) and red arrows mark the actual fill order (inconsistent).

Hi, i have a question, if a have 15 swiper-slides elements  want 3 slidesPerView and 3 slidesPerColumn,
how i can order these elements into a 2 Views each one with  9 slides grid? thanks.
Example:
1|2|3                 10|11|12
4|5|6   -------> 13|14|15
7|8|9

I have found a solution to this problem for my project. The problem seems to occur because of how CSS flexbox works with Swiper. It still fills the slides by row, and column is only simulated by using CSS order, so the last remaining items always show on the last row, instead of in the last column.
My solution is to calculate how many slides are missing to fill the whole grid, then append placeholder empty slides equal to number of missing slides to Swiper on init, and then Swiper will calculate and fill the columns correctly.
Code I’m using is here:
onInit: function (swiper) {
    // Slides per column
    console.log('slidesPerColumn: ' + swiper.params.slidesPerColumn);
    // Total number of slides
    console.log('totalNumberOfSlides: ' + swiper.slides.length);
    // Slides outside even grid
    var slidesOutsideEvenGrid = swiper.slides.length % swiper.params.slidesPerColumn;
    console.log('slidesOutsideEvenGrid: ' + slidesOutsideEvenGrid);
    // Check if grid is totally filled
    if (slidesOutsideEvenGrid != 0) {
        console.log('not even grid');
        // Grid not totally filled, calculate slides needed to fill grid
        var slidesNeededToFillGrid = swiper.params.slidesPerColumn - slidesOutsideEvenGrid;
        console.log('slidesNeededToFillGrid: ' + slidesNeededToFillGrid);
        // Append as many empty placeholder slides needed to fill grid
        for (var i = 0; i < slidesNeededToFillGrid; i++) {
            swiper.appendSlide('

');
        }
    }
}

Hope something like this can be implemented in core to solve the problem with filling columns correctly.
Live example can be found here: https://www.myresjohus.se/inspiration/hall/
(At the time this comment was written, the code above was not yet deployed for the live example above, so if order is still wrong for the thumbnails in the Swiper slideshow, check back at the live example in a couple of days to see it working)


Update d26e6786-064a-174c-5b9f-79e85b34f59b.md

Passing a Range object as SourceData seems to be a known issue or several years that has not been fixed even in Office 2013.
There are several stack overflow discussions where people who have passed Range objects to PivotCache.Create get a "type mismatch" error unexpectedly, even after the code ran successfully in previous testing. I encountered this where workbooks I had created with a Range object worked fine for weeks of testing, and then failed with "type mismatch" sporadically after the workbooks were put into circulation. Using a string as a workbook/cell range reference or a named range as a reference appeared to fix the problem. The official documentation should be updated, because passing a Range object here is unstable, yet the documentation specifies that is what should be passed.

Hi @thetriggestcove, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


Documentation, how to send a dynamic update with/without TSIG

Dear, Sirs!
Could you please update the documentation to include
comprehensive and complete examples of how send a dynamic
update with/without TSIG?
I have been trying with the API for several days, but I don't understand
how to do this.
This would be very much appreciated!
Thank you!

what have you tried, where did you get stuck, what code didn't work?

Hello, Sir!
Since I was completely ignorant regarding dynamic updates of DNS, I was in deep water already in the beginning.  I read the documentation many times, but because there was not a complete example of how to add, update or delete DNS-records I got very confused.  A colleague of mine also insisted that I needed to use IXFR-call(s) to do these operations, which also confused me even more.
After a week I discovered that I could find examples in update_test.go, which were exactly what I needed.  Then everything fell into place.
Anyhow, I believe it would be a generally good idea to at least include one complete example of how to do dynamic update in the code-examples.  The idea is to get users of the library, who are not bleeding experts on DNS and the belonging protocol up to speed faster.  Many programmers, like myself, jump from project to project, and sadly we are not always able or have the time to examine and understand all the underlying details.
I hope you appreciate the idea.  Thank you very much!

I don't know if that would work. The relevant RFC tell you how to create dns update
messages; that should easily translate into this library.
/close

https://github.com/StackExchange/dnscontrol/pull/729/files#diff-1e47ac91edeaba741d1ddfca731a7457044db3df4ef2216a5018f319f9c85af1R339-R378 shows how to implement it.


api.route("") returns Not Found

When an empty route is provided it returns 404
@api.route("")
def test(req, resp):
    resp.text += "test"

I'm gonna write a quick patch to fix that.
-//
@kennethreitz is it time to add some append_slashes param  ?

I know, but what I'm talking about it here is to automatically append or remove slashes.

oh ok

I think we're good as-is.


fix(lld): fix auto update to not have a "Download update" button

📝 Description
on LLD, the auto update mechanism have an edge case that makes it fail, typically when you click on the Download update button.
instead of "Download update" we want to let Electron's autoDownload to do it in background.
the fact we had both autoDownload AND downloadUpdate() logic was creating a race condition which make the update fail. as described by https://github.com/electron-userland/electron-builder/issues/3622#issuecomment-578663256
essentially, this downloadUpdate() is a forgotten code from a past feature we have sunset since then (the ability to manually do the update, we moved to autoDownload since that time)
❓ Context

JIRA or GitHub link: https://ledgerhq.atlassian.net/browse/LIVE-10212

✅ Checklist
Pull Requests must pass the CI and be code reviewed. Set as Draft if the PR is not ready.

[x] npx changeset was attached.
[ ] Covered by automatic tests. only partially covered (some mocked parts)
[x] Impact of the changes:

LLD Auto Update

🧐 Checklist for the PR Reviewers

[ ] The code aligns with the requirements described in the linked JIRA or GitHub issue.
[ ] The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
[ ] There are no undocumented trade-offs, technical debt, or maintainability issues.
[ ] The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
[ ] Any new dependencies have been justified and documented.
[ ] Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

/generate-screenshots


Tell me to set SDK to >=2.13, "This requires the 'nonfunction-type-aliases' language feature to be enabled"

error: This requires the 'nonfunction-type-aliases' language feature to be enabled. (experiment_not_enabled at [json_serializable] test/type_defs/typedef_usage.dart:9)

Once a feature is stable, we should point folks to the version of the SDK they need to set in their pubspec.
The language feature verbiage is not helpful.

That message is coming from the parser.

The tip associated with this error actually says
Try updating your pubspec.yaml to set the minimum SDK constraint to 2.13 or higher, and running 'pub get'.

but it is not emitted by the analyzer.
@bwilkerson Should we make the analyzer emit the tip as well?

Should we make the analyzer emit the tip as well?

The analyzer already emits this tip.
@kevmoo Where are you seeing this message?
The dart analyze command currently produces the following:
  error • test.dart:1:11 • This requires the 'nonfunction-type-aliases' language feature to be enabled. Try updating your
          pubspec.yaml to set the minimum SDK constraint to 2.14.0 or higher, and running 'pub get'. •
          experiment_not_enabled

while the dartanalyzer command produces
  error • This requires the 'nonfunction-type-aliases' language feature to be enabled. • test.dart:1:11 • experiment_not_enabled

neither of which match the output you provided.

@bwilkerson – I was seeing it in IntelliJ

That's an IntelliJ presentation issue. The correction message was sent to the client and the client displays it in some places (try hovering over the diagnostic in the Dart Analysis view or in the editor), but it appears that it wasn't being displayed in a place that you discovered.
The question is, where would you have expected to have seen it?

@bwilkerson – I would hope the line that shows up in the IDE to point to the language version
CC @devoncarew and @jwren

Possibilities that I can think of include

adding a new column in the Dart Analysis view's table,
appending the correction to the message under "Description",
making the rows multiple lines long and adding the correction on a second line, or
making the diagnostics expandable (tree table) and putting the correction in as a child of the diagnostic (possibly with the context messages as children as well).

Could we change the content of the primary message? It's not helpful at all...

Probably. That's coming from the Parser, so that would be front-end work. Do you have a suggestion?

@kevmoo Do you have a suggestion for what wording would have helped in your situation?

FWIW, the non-parser messages produced by the analyzer have the form:

Extension methods weren't supported until version 2.6.0, but this code is required to be able to run on earlier versions.

Rather than using the experiment name it focuses on the version number (for features that have been released, of course).

@johnniwinther  "This requires Dart X.Y. Please see dart.dev/x/y/z for more information."

I like the message Kevin suggests, but think we should link to https://dart.dev/guides/language/evolution#language-versioning instead (we can create a shorter go-link if desired).

Most messages related to (experimental) features are not generated in the parser but in the ast builders in the analyzer and cfe, respectively, where the context of the library language version is available. The problem with the parser generated messages is that they are somewhat hardcoded and therefore easily go stale. I'm working on a CL to mitigate this, but I'm not sure that I can currently produce the messages mentioned in this issue (anymore?).
@kevmoo Can you create repros for the problems you've seen?

Absolutely, @johnniwinther – just use library; with latest dev in a package that has sdk: >=2.17.0 <3.0.0 or similar

error: This requires the 'unnamed-libraries' language feature to be enabled. (experiment_not_enabled at [http2] lib/src/async_utils/async_utils.dart:7)

with Dart SDK version: 2.19.0-361.0.dev (dev) (Tue Nov 1 15:30:40 2022 -0700) on "macos_arm64" in intellij

@kevmoo Do you know how you got the error related to nonfunction-type-aliases in the original issue? The error related to unnamed-libraries is provided by the ast builder and not the parser. (The message needs to be updated, though).

...that might have been fixed. 🤷  I just piled on here because I hit the same flavor of issue w/ the latest SDK (and the "naked" library feature)

Yeah I think it would be super beneficial to just slip the version number into the primary message. Instead of:

This requires the 'nonfunction-type-aliases' language feature to be enabled.

Something like

This requires a minimum Dart Language version of '2.13.0', which introduces the 'nonfunction-type-aliases' language feature.

And we should have a quick fix as well (do we?) so that users don't have to go read documentation for something that's really not that interesting; knowledge about Dart SDK versions vs Dart language versions hopefully doesn't need to be top-of-mind for the average developer.


documented requestBody

This PR adds documentation for requestBody to describe inputs into operations.
This fixes #665, #303, #222, #418, #179, #874, #800, #303,
It clarifies #430

There are a few places where you use requestBody and I wonder what you think about using a link to the "Request Body Object" documentation itself instead?  I see that in the "Content Object" section _(https://github.com/OAI/OpenAPI-Specification/blob/OpenAPI.next/versions/3.0.md#content-object) you're already doing this and that's why it came to mind.  Other than addressing this for consistency, LGTM.

Thanks for putting this together, @fehguy - this addresses a lot of important issues.
Following our discussions, one thing is missing and that's addressing custom serialization format with x-www-form-urlencoded. The defaults are mentioned. IIRC, these should fall under the Encoding Object.
If we choose to skip that, then the Encoding Object structure can be simplified.

LGTM (though Content-Type should be contentType).

LGTM (though see comment about content-type, Ron might be wrong!)

@OAI/tdc shall we merge? (cha cha cha)

LGTM

:shipit:


fix(chart): add prometheusrules in clusterrole

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #815

License
Apache 2.0

What's in this PR?
logging operator 3.14 introduced the possibility to create default prometheus rules. In consequence, the chart need to deploy adapted clusterrole.
Checklist

[x] Related Helm chart(s) updated (if needed)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

this is going to be fixed in https://github.com/banzaicloud/logging-operator/pull/818
(the go annotation had to be changed, rbac resources are generated)


Replace index with navbar

Replace index page with a navbar of links to all files that are generated by my simple site generator.

closed by 7435a

closed by 7435a


[hotfix][doc]There is a link redirection problem in the Chinese document of ‘custom_serialization’

What is the purpose of the change
There is a link redirection problem in the Chinese document of ’Custom Serialization for Managed State‘.
When I wanted to click on the link of 'type serialization framework' in the Chinese document to get more information, the page jumped to 404, but when I checked the English document, I found that turn to correct page. So i fix the link of 'type serialization framework' in the Chinese document.

Brief change log

Modify the link of 'type serialization framework' in custom_serialization.md which under content.zh turn to 404 page.

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no) no
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no) no
The serializers: (yes / no / don't know) no
The runtime per-record code paths (performance sensitive): (yes / no / don't know) no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know) no
The S3 file system connector: (yes / no / don't know) no

Documentation

Does this pull request introduce a new feature? (yes / no) no
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented) no

Hi, @MartijnVisser  There is another wrong link in the document of 'Custom_serialization'.  I'm sorry to disturb you, but can you help me review it when you are free?


varnishPackages: general version bumps to fix CVE-2020-11653

Motivation for this change
Fixes CVE-2020-11653.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review run on x86_64-linux 1

6 packages built:

collectd
collectd-data
prometheus-varnish-exporter
varnish
varnish60
varnish62

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110892 run on x86_64-darwin 1

2 packages marked as broken and skipped:

collectd
collectd-data

4 packages built:

prometheus-varnish-exporter
varnish (varnish63)
varnish60
varnish62

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110892 run on x86_64-darwin 1

2 packages marked as broken and skipped:

collectd
collectd-data

4 packages built:

prometheus-varnish-exporter
varnish (varnish63)
varnish60
varnish62

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110892 run on x86_64-linux 1

6 packages built:

collectd
collectd-data
prometheus-varnish-exporter
varnish (varnish63)
varnish60
varnish62

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110892 run on x86_64-linux 1

6 packages built:

collectd
collectd-data
prometheus-varnish-exporter
varnish (varnish63)
varnish60
varnish62


[doc] feat: ADR for implementation of fetching DMAP data

For reference: Asana
This PR introduce an ADR document for how to implement the fetching of DMAP data.

@paulswartz For the past few days, I've done some work to make sure this approach is feasible. I'm presenting it here as an ADR. Let me know what you think please.


Better error message when Nginx config is not found

Hey!
Right now I'm getting the following error when I run rhit without Nginx set up:
Error: Io(Os { code: 2, kind: NotFound, message: "No such file or directory" })

I think we can make this error message more informative.
I would like to submit a PR if applicable 🐻

I would like to submit a PR if applicable

It would probably be welcome.


Add support for --token-usage flag, issue #41

Description
This PR introduces a new --token-usage flag to the CLI, which allows users to output token usage information when interacting with the Gemini model. The changes are implemented in multiple files across the project to ensure that the flag is correctly handled, and the token usage statistics are printed to stderr if the flag is passed.
Changes made:

_main.py

Added the --token-usage flag using typer.Option. This flag is a boolean and defaults to False. When passed, it enables printing the token usage statistics.

token_usage: Annotated[
    bool, typer.Option("--token-usage", help="Flag for printing token usage")
] = False

application/core/gemini_model.py

Modified the get_gemini_summary function to return both the formatted summary of the GitHub repository and the token usage metadata. The return type was updated from str to Dict[str, str] to accommodate both the formatted response and usage metadata.

Original function:
def get_gemini_summary(github_data: Dict[str, Any], model_temperature: float) -> str:
#...
return formatted_response

Updated function:
def get_gemini_summary(github_data: Dict[str, Any], model_temperature: float) -> Dict[str, str]:
#...
return {'formatted_response': formatted_response, 'usage': response.usage_metadata}

application/utils/callbacks.py

Implemented logic to handle the --token-usage flag. If the flag is set, the token usage statistics (prompt tokens, completion tokens, and total tokens) are printed to stderr in a formatted, colored output.

Added code:
if token_usage:
    formatted_usage = ("\n\033[92m"
                       "Token Usage:\n"
                       "-------------\n"
                       f"- Completion Tokens: {usage.candidates_token_count}\n"
                       f"- Prompt Tokens: {usage.prompt_token_count}\n"
                       f"- Total Tokens: {usage.total_token_count}\n"
                       "\033[0m")
    err_console.print(f"\n[green]{formatted_usage}[/green]\n")

Reason for Changes
The addition of this flag provides greater transparency to users regarding the token usage when using the Gemini model. This can be helpful for optimizing token counts and understanding the underlying resource usage during model interactions.

Why --token-usage: This flag allows users to explicitly request token statistics, avoiding clutter in standard output for users who do not need this information.
Design choice: I chose to return the usage metadata as part of the existing get_gemini_summary function to keep the return structure compact and useful for multiple purposes.

Challenges or Bugs

No bugs encountered during testing.

This PR improves the usability of the Gemini model CLI and adds a useful feature for users who want to monitor token consumption during usage.

I have tested the changes and they all look great! Thank you so much!


RGB-D data collection in real world

Hi,
Thank you for sharing this great work! I notice that you use Fast-LIO for localization which is very smooth in reconstructing the real-world building. I also try to use Fast-LIO from the official repo however the performance of the localization is not that good. Could you provide details about your modified version of Fast-LIO? Thanks!!

Hello,
We used FAST_LIO_LC to create the map of the real-world building, (get the RGBD sequence to create the graph).
For the navigation tasks we used the real-world map PointCloud (FAST-LIO-LC) + Spot robot map (navigation stack) for the same building, to navigate autonomously.
Our method can work with any SLAM system as long it accurate enough + Robot with a Navigation stack.
You can take a look here:
https://github.com/yanliang-wang/FAST_LIO_LC
https://github.com/gisbi-kim/FAST_LIO_SLAM
https://github.com/HViktorTsoi/FAST_LIO_LOCALIZATION


[Feature]: Configuration file for Letterboxd (to allow login)

Is your feature request related to a problem? Please elaborate.
Letterboxd has the option to make a list Private, so only the creator can view it. Unfortunately the way they do it seems to involve blocking the URL for all but the logged in creator.
I surmise that because PMM errors out when trying to pull from a list marked Private:
Value: ['https://letterboxd.com/markmarz/list/classic-period-crossover-films-noir-westerns/', 'https://letterboxd.com/markmarz/list/classic-period-crossover-films-noir-westerns-1/'] |
| Traceback (most recent call last): |
| File "//plex_meta_manager.py", line 543, in run_collection |
| builder = CollectionBuilder(config, metadata, mapping_name, collection_attrs, library=library, extra=output_str) |
| File "/modules/builder.py", line 689, in init |
| self._letterboxd(method_name, method_data) |
| File "/modules/builder.py", line 1133, in _letterboxd |
| letterboxd_lists = self.config.Letterboxd.validate_letterboxd_lists(self.Type, method_data, self.language) |
| File "/modules/letterboxd.py", line 87, in validate_letterboxd_lists |
| raise Failed(f"{err_type} Error: {final['url']} failed to parse") |
| modules.util.Failed: Collection Error: https://letterboxd.com/markmarz/list/classic-period-crossover-films-noir-westerns-1/ failed to parse |
| |
| Collection Error: https://letterboxd.com/markmarz/list/classic-period-crossover-films-noir-westerns-1/ failed to parse |
But succeeds pulling from the same URL when the list is marked Public.
I'm thinking if there was a way to login to Letterboxd, this problem may be solved.
Thanks!
Describe the solution you'd like
Add configuration file for Letterboxd to allow login, thus enabling fetch of Private Letterboxd lists.
Additional Information
No response

API here:
https://api-docs.letterboxd.com/
BUT:
https://letterboxd.com/api-beta/
"The Letterboxd API is in private beta"

API here: https://api-docs.letterboxd.com/
BUT: https://letterboxd.com/api-beta/ "The Letterboxd API is in private beta"

Thanks! That's very helpful information.

Feature Requests have been moved to https://features.metamanager.wiki.
This Request


http: server gave HTTP response to HTTPS client

Description
I am trying to setup a local insecure repository on VM running Ubuntu 16.04.1 image. I have followed official configuration guide for local registry and created a daemon.json placed in /etc/docker/daemon.json.  I have also modified the contents of /etc/default/docker according to manual. Unfortunately, no luck with plain HTTP repository. Help!
daemon.json
{
    "insecure-registries": ["<IP_ADDRESS>:5000"]
}

Command
I am attempting to connect from a different host (<IP_ADDRESS>, with full access permissions for <IP_ADDRESS>) and push to <IP_ADDRESS>:5000.
docker push <IP_ADDRESS>:5000/peptone-rpc-controller

Error:
The push refers to a repository [<IP_ADDRESS>:5000/peptone-rpc-controller]
Get https://<IP_ADDRESS>:5000/v1/_ping: http: server gave HTTP response to HTTPS client

docker info
Containers: 2
 Running: 1
 Paused: 0
 Stopped: 1
Images: 1
Server Version: 1.12.3
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 19
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: host null bridge overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.4.0-47-lowlatency
Operating System: Ubuntu 16.04.1 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 488.3 MiB
Name: peptone-registry
ID: EK2K:QGNT:4WSX:XFZ3:2LX6:2BR3:LUYV:2U66:4OCF:O4TZ:ZXWO:PY3X
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
 <IP_ADDRESS>:5000
 <IP_ADDRESS>/8

/etc/default/docker
# Docker Upstart and SysVinit configuration file

#
# THIS FILE DOES NOT APPLY TO SYSTEMD
#
#   Please see the documentation for "systemd drop-ins":
#   https://docs.docker.com/engine/articles/systemd/
#

# Customize location of Docker binary (especially for development testing).
#DOCKERD="/usr/local/bin/dockerd"

# Use DOCKER_OPTS to modify the daemon startup options.
DOCKER_OPTS="--insecure-registry <IP_ADDRESS>:5000"

# If you need Docker to use an HTTP proxy, it can also be specified here.
#export http_proxy="http://<IP_ADDRESS>:3128/"

# This is also a handy place to tweak where Docker's temporary files go.
#export TMPDIR="/mnt/bigdrive/docker-tmp"

restart your docker daemon in debug mode (add -D to the daemon launch arguments)

Can you please provide your daemon debug logs while reproducing this error.

Absolutely!
I attach the output from journalctl -xe | grep dockerd upon the dockerd restart with --debug flag.
Nov 11 21:54:09 peptone-registry dockerd[998]: time="2016-11-11T21:54:09.413674939Z" level=info msg="Processing signal 'terminated'"
Nov 11 21:54:09 peptone-registry dockerd[998]: time="2016-11-11T21:54:09.556399619Z" level=warning msg="libcontainerd: container 13356383fa407269567359c7919e4a1967ad25e8b51a173f3da8030ba20b4b5b restart canceled"
Nov 11 21:54:10 peptone-registry dockerd[998]: time="2016-11-11T21:54:10.134761269Z" level=info msg="stopping containerd after receiving terminated"
Nov 11 21:54:11 peptone-registry dockerd[1650]: time="2016-11-11T21:54:11.390568384Z" level=info msg="libcontainerd: new containerd process, pid: 1656"
Nov 11 21:54:12 peptone-registry dockerd[1650]: time="2016-11-11T21:54:12.785624105Z" level=info msg="[graphdriver] using prior storage driver \"aufs\""
Nov 11 21:54:12 peptone-registry dockerd[1650]: time="2016-11-11T21:54:12.832667912Z" level=info msg="Graph migration to content-addressability took 0.00 seconds"
Nov 11 21:54:12 peptone-registry dockerd[1650]: time="2016-11-11T21:54:12.844202370Z" level=warning msg="Your kernel does not support swap memory limit."
Nov 11 21:54:12 peptone-registry dockerd[1650]: time="2016-11-11T21:54:12.851264433Z" level=info msg="Loading containers: start."
Nov 11 21:54:12 peptone-registry dockerd[1650]: ..time="2016-11-11T21:54:12.946525439Z" level=info msg="Firewalld running: false"
Nov 11 21:54:13 peptone-registry dockerd[1650]: time="2016-11-11T21:54:13.607097355Z" level=info msg="Default bridge (docker0) is assigned with an IP address <IP_ADDRESS>/16. Daemon option --bip can be used to set a preferred IP address"
Nov 11 21:54:13 peptone-registry kernel: aufs au_opts_verify:1597:dockerd[1665]: dirperm1 breaks the protection by the permission bits on the lower branch
Nov 11 21:54:14 peptone-registry kernel: aufs au_opts_verify:1597:dockerd[1665]: dirperm1 breaks the protection by the permission bits on the lower branch
Nov 11 21:54:15 peptone-registry dockerd[1650]: time="2016-11-11T21:54:15.350024203Z" level=info msg="Loading containers: done."
Nov 11 21:54:15 peptone-registry dockerd[1650]: time="2016-11-11T21:54:15.354939963Z" level=info msg="Daemon has completed initialization"
Nov 11 21:54:15 peptone-registry dockerd[1650]: time="2016-11-11T21:54:15.361219981Z" level=info msg="Docker daemon" commit=6b644ec graphdriver=aufs version=1.12.3
Nov 11 21:54:15 peptone-registry dockerd[1650]: time="2016-11-11T21:54:15.509260523Z" level=info msg="API listen on /var/run/docker.sock"

Can you just send the logs from the period in which you reproduced the error. The logs don't include the push attempt.

I can only see journal journalctl on the remote (<IP_ADDRESS>) machine from which I want to push (to <IP_ADDRESS>).
Nov 11 23:08:13 peptone dockerd[14445]: time="2016-11-11T23:08:13.763732262+01:00" level=error msg="Attempting next endpoint for push after error: Get https://<IP_ADDRESS>:5000/v2/: http: server gave HTTP response to HTTPS client"
Nov 11 23:08:13 peptone dockerd[14445]: time="2016-11-11T23:08:13.768440062+01:00" level=error msg="Attempting next endpoint for push after error: Get https://<IP_ADDRESS>:5000/v1/_ping: http: server gave HTTP response to HTTPS client"

There is no response in the logs on the registry machine (<IP_ADDRESS>).

@dmcgowan I have found the issue. The docker client needs to have insecure registry set via daemon.json. The modification to /etc/default/docker seems not to work on Ubuntu 16.04.1 / docker 1.12.3.

@ktamiola

I have found the issue.The docker client needs to have insecure registry set via daemon.json

But you have already configured this earlier as you mentioned

I am trying to setup a local insecure repository on VM running Ubuntu 16.04.1 image. I have followed official configuration guide for local registry and created a daemon.json placed in /etc/docker/daemon.json

the directory /etc/default/docker doesn't exist for me
What configuration did you change to make it work? I'm running into similar issue on Docker 17.09

Just need to restart the docker daemon after the changes in daemon.json

No, only docker service restart worked for me.


[retire] Reire page post launch enhancements

@ladytrekker here is a collection of fixes / enhancements to the Retire page

[ ] Hover state on project cards on the Retire page should match hover state on the /projects cards. Currently they are reversed, i.e. when I hover on a card on the Retire page the focused card shrinks slightly in size, vs expanding.

[x] For use case where a user has logged in at least once but does not have a Carbonmark profile, update the design to match below. Figma design link here.

[ ] Featured project card descriptions should match design and be immediately following the project title, like cards in /projects:

@jabby09 Point 2 is already integrated.
Just create a new wallet address, register on carbonmark, go to the Retire page

UPDATE: Sorry, you are talking about the other state

exactly :)


Swap vertical/horizontal variables in row/column calculation

Currently has no impact as all code-sizes are square, but if support for rectangular DataMatrix would be added, then this would have been an issue.
(I might try to add that support, but I don't know Go 🙂)

Thank you for contributing


test anchor redirects

@ggailey777,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


I tried generating bevel gear in fusion and it failed.

The error message was prompted that there are too much constraint.

See https://github.com/padarom/bevel-gear/issues/1#issuecomment-701582013. This add-in is still a work-in-progress and not yet fully usable.


Enable click through target in Kotlin

Based on @shurwit pull request:
These changes allow you to enable the ability to pass touch events through the target part of the spotlight while maintaining the close on touch outside functionality on the overlay.

Hi, thank you for the PR.
As of the version of 2.0.0, touch outside functionality was removed so you can easily click the views under SpotlightView because SpotlightView doesn't consume the touch event anymore.
If you want to archive touch outside functionality, you can do that by adding a clickListener and call Spotlight.close() inside.
I'm closing thie PR since views under Spotlight are clickable for now.
Again, thank you for your time and suggestion. I appreciate it.


Update all schemas to be valid file URIs

User Story
As a report stream developer, I would no longer like to use relative paths while resolving schemas
Description/Use Case
This captures the work of updating all the schemas in source code and replacing relative paths with file URIs
Risks/Impacts/Considerations
Dev Notes
Acceptance Criteria

[ ] None of the schemas reference relative paths

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @brick-green @jack-h-wang @JessicaWNava @JFU-NAVA-PBC @jimmyfagan @thetaurean

@mkalish Please check with Jimmy/Arnej on follow up questions.

Hey team! Please add your planning poker estimate with Zenhub @arnejduranovic @jack-h-wang @JFU-NAVA-PBC @luis-pabon-tf @mkalish @thetaurean


Stop searching when it finds the first difference

Sometimes it takes a lot of time to get isEqual result when the files are too different.
I think it's safe to return isEqual = false on first difference found.

Hi!
At the moment, the tool is not actively developed, but pull requests are wellcome.

Hi!
At the moment, the tool is not actively developed, but pull requests are wellcome.


Error message when re-opening termdebug

I get an error in the following scenario. Debugging seems to work still, so it's not breaking, but it does break my flow.
Scenario:

Open vim
':Termdebug'
Enter 'q' in the !gdb buffer
':Termdebug'
Load some symbols ('file /tmp/foo')
Open source file
':Break'

Result: error message:
E121: Undefined variable: s:BreakpointSigns                                                                                E116: Invalid arguments for function index(s:BreakpointSigns, a:nr) == -1                                                                                E15: Invalid expression: index(s:BreakpointSigns, a:nr) == -1                                                                                Error detected while processing function <SNR>144_CommOutput[12]..<SNR>144_HandleNewBreakpoint[20]..<SNR>144_PlaceSign:                                                                                line    1:                                                                                E155: Unknown sign: debugBreakpoint1
Version info:
VIM - Vi IMproved 8.1 (2018 May 17, compiled Jun 20 2018 02:44:05)
Included patches: 1-89

I get an error in the following scenario. Debugging seems to work
still, so it's not breaking, but it does break my flow.
Scenario:

Open vim
':Termdebug'
Enter 'q' in the !gdb buffer
':Termdebug'
Load some symbols ('file /tmp/foo')
Open source file
':Break'

Result: error message:
E121: Undefined variable: s:BreakpointSigns E116: Invalid arguments for function index(s:BreakpointSigns, a:nr) == -1 E15: Invalid expression: index(s:BreakpointSigns, a:nr) == -1 Error detected while processing function <SNR>144_CommOutput[12]..<SNR>144_HandleNewBreakpoint[20]..<SNR>144_PlaceSign: line    1: E155: Unknown sign: debugBreakpoint1
Version info:
VIM - Vi IMproved 8.1 (2018 May 17, compiled Jun 20 2018 02:44:05)
Included patches: 1-89

I cannot reproduce this.  Please get at least version 8.1.0100.
--
Google is kind of like Dr. Who's Tardis; it's weirder on the
inside than on the outside...
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I cannot reproduce it in 8.1.0178, sorry for the trouble.


Minor change to local target in the makefile

Should it  be '{print $2}'  instead of  '{print $$2}' in the local target?

In a makefile the $ needs to be escaped as $$.
Refer to https://www.gnu.org/software/make/manual/html_node/Variables-in-Recipes.html


Add close method

Addresses #21 and #25 by adding a stub for the close method which simply resolves a promise. This fixes calling Sentry.flush which was throwing:
TypeError: _this._getBackend(...).getTransport(...).close is not a function

@randallagordon sorry I haven't merge it :/
We're now doing some refactoring work with this, can you pls rebase and resolve conflicts?

@zivl sure thing! I'll try to get to it in the next couple days.

@zivl Should be squared away! Just had to pepper some .originalReports around in our local test suite thanks to the v2->3 changes, but otherwise drops straight into our test suite—no more local monkey-patching! 💥

@randallagordon job well done!
🎉

published in v3.0.2

resolves #21 and resolves #25


Build binary for ppc64le platform

Build the binary package for ppc64le platform

closing this on behalf of https://github.com/heptio/velero/pull/1605


Add More Email Recipient Options

Currently:
'send_to' => array('assignee' => t('Send to Assignee'), 'creator' => t('Send to Creator'), 'both' => t('Send to Both')),

Make this:

Send To:

Assignee (exists as assignee)
Creator (exists as creator)
Assignee & Creator (exists as both)
Project Email
Assignee & Project Email
Creator & Project Email
All (Assignee, Creator, Project Email)

Relates to #4

'send_to' => array('assignee' => t('Send to Assignee'), 'creator' => t('Send to Creator'), 'both' => t('Send to Both')),

Possible Options:
'send_to' => array(
    'assignee' => t('Send to Assignee'),
    'creator' => t('Send to Creator'),
    'both' => t('Both Assignee'),
    'project_email => t('Project Email'),
    'assignee_project_email' => t('Assignee & Project Email'),
    'creator_project_email') => t('Creator & Project Email'),
    'all' => t('Assignee, Creator & Project Email')
    ),


archwsl: Don’t erase file system on upgrades

Fixes #3800 (i haven’t tested it, but it should work)

Looking forward to this! I was extremely lucky that I backed up my heavily modified Arch instance before it got wiped in an update.

Can someone test this fix, and confirm it works?

Can someone test this fix, and confirm it works?

If do a force update will be sufficient to test, I can backup my Linux and  try it now.

Well it still don't work as expected, because we haven't changed the post_install script where in line 32 arch install $dir\rootfs.tar.gz\ is still invoked unconditionally. Look this up in arch --help or [archwsl offcial wiki](https://github.com/yuk7/ArchWSL/wiki/How-to-Use(for-Installed-Instance), we can see it will restore from a tar.gz backup. In this case it the initial clean rootfs and thus cover our data.
I suggest that in the post_install script (which will run after install and update) we also bypass the execution of arch install on $cmd -eq update.

Well, it still don't work.
Here is what I tried:
scoop uninstall archwsl
cd ~\scoop\buckets\extras\bucket\
git restore -s db31afbe25803b9ae7ca88176d57b6ba86162156~ archwsl.json
scoop install archwsl
arch

touch a
useradd -m myaccount
exit

arch config --default-user myaccount
arch

touch b

git reset --hard
scoop update archwsl
arch

ls -l
ls /home

At this moment, what happened is the default user is back to root and files a and b and user myaccount don't exist.
I'm still trying to find a possible solution without touching the code of Arch.exe right now...

@rasa It works, but other users should be aware that you have to manually insert this change into your old, current version of archwsl's manifest.json, which should be ~\scoop\apps\archwsl\<IP_ADDRESS>\manifest.json before doing the update. Or the old uninstaller script will still wipe your ArchLinux data.
 The first is that why not use if ($cmd -eq 'update') { return }? I personally think that's more descriptive about its concern and precise. Maybe more future-proof, too.
The other is that because that we didn't add a symmetric check-and-bypass in the post_install script, so that script will result in an (not severe but still shouldn't be ignored) error like this:
scoop update archwsl

archwsl: <IP_ADDRESS> -> <IP_ADDRESS>
WARN  Scoop uses 'aria2c' for multi-connection downloads.
WARN  Should it cause issues, run 'scoop config aria2-enabled false' to disable it.
Updating one outdated app:
Updating 'archwsl' (<IP_ADDRESS> -> <IP_ADDRESS>)
Downloading new version
Loading Arch.zip from cache.
Checking hash of Arch.zip ... ok.
Uninstalling 'archwsl' (<IP_ADDRESS>)
Running uninstaller script...
Removing shim for 'Arch'.
Unlinking ~\scoop\apps\archwsl\current
Installing 'archwsl' (<IP_ADDRESS>) [64bit]
Loading Arch.zip from cache.
Extracting Arch.zip ... done.
Linking ~\scoop\apps\archwsl\current => ~\scoop\apps\archwsl\<IP_ADDRESS>
Creating shim for 'Arch'.
Creating shortcut for Arch Linux (Arch.exe)
Persisting data
Running post-install script...
ERROR: This instance is already installed.
If you want to reinstall it, you can remove it with the "clean" command.
ERROR Exit code was -2147467260!
ERROR ArchWsl installation failed!
'archwsl' (<IP_ADDRESS>) was installed successfully!
Notes
-----
Even when you are logging in as 'root', some operations (like service command) require Windows administrator privileges

I suggest we also add the check for post_install so it prevents us from potential problems caused by changes to arch install, and avoid the unnecessary copy in the post_install script, etc.